Code review comment for lp:~lsinger/play/1.1

Revision history for this message
Leif Singer (lsinger) wrote :

> Ok merged. Another cool feature we could provide in this room is:
>
> protected static void waitFor(Future tasks...) {
> }
>
> It could wait for several tasks completions before resuming the request.

Thanks a lot for merging so quickly! It's nice to have own code in my favorite server-side framework. :)

As for the waitFor for multiple Futures: it looks like it would be useful for some, but then again, I've actually never worked with Futures, so I couldn't even tell whether it works correctly. So I guess I should leave that one to you to decide.

Regards
  Leif

« Back to merge proposal