Follow-up from "[MINOR] Initial Queue Estimates"
The following discussion from !5048 (closed) should be addressed:
-
@MSevey started a discussion: (+1 comment) nit: Should this be a constant?
errWorkerFailedElapseTime
or something?Also should this be checking the value of
elapsed
in addition? An error can be returned afterelapsed
was already set, but it could have taken longer than 1 second, in which case we would be incorrectly improving the worker stats right?