#1433
  • Created on
    2011-12-05 15:29
  • Reported by
    viktorklang
  • Status
    Fixed
  • Priority
    Highest (1)
  • Assigned to
    viktorklang
  • Milestone
    2.0-M1
  • Plan Level
    -
  • Due Date
    -

Attachments

Related Tickets

Add new subtask or bug

Remove timeouts from Futures

Needlessly complicates things, add sugar for:

scheduler.scheduleOnce(X time, promise completeWithException new FutureTimeoutException(...) with NoStackTrace)
User picture
  on 2011-12-05 19:41 *
By Derek Williams
It might be a good idea to also cancel the scheduled task when the promise is completed so the promise can be garbage collected sooner. I don't think it's a big problem, but it might be something to think about.
User picture
  on 2011-12-06 11:00 *
By viktorklang
Yeah, that's a very good idea Derek! It'd be crazy simple to do also, and you only pay the price if you actually use timeouts. What are your thoughts on the idea as a whole? Would love to hear what you think.
User picture
  on 2011-12-08 22:25 *
By viktorklang
Priority changed from Normal (3) to Highest (1)
User picture
  on 2011-12-08 22:38 *
By viktorklang
Assigned to set to viktorklang
Status changed from New to Accepted
User picture
  on 2011-12-14 00:58 *
By viktorklang
Status changed from Accepted to Test
User picture
  on 2011-12-14 16:45 *
By viktorklang
Status changed from Test to Fixed