Hi, On Fri, Jan 10, 2014 at 3:57 PM, Carsten Ziegeler <[email protected]> wrote: > ...the interesting case still is b) - I see the point that we return WARN > in that case - I would assume that getElapsedTimeInMs returns -1 and > getFinishedAt() returns null. So both could be used as an indicator for > this situation....
Turns out that I had an unsent draft on this, sorry...I agree in principle, WARN is ok to indicate "no result available". I'd return the epoch (new Date(1L) ?) for getFinishedAt() instead of null which requires you to check for null everywhere you use that. Returning
