Edit, I think I might know why, because that's what was done in a couple of other places.  Sorry those were probably just leftover code from when I was still using deferred results for everything before I changed it over to only be deferred results where it made sense.  What you're doing here shouldn't require that so you should be able to remove `NOT_DONE_YET`