joshelser commented on pull request #3667:
URL: https://github.com/apache/hbase/pull/3667#issuecomment-920397705


   > there are some limitations in MonitoredTask which doesn't allow me to 
track start time and end time for each state. Also it doesn't show the states 
that are completed. Since MonitoredTask is being used at many places, I didn't 
want to change just for HMaster startup use case. WDYT ?
   
   IMO, if start/end time is the only thing you're missing, then just add that 
to MonitoredTask and adapt your UI to use that class instead. I think that 
Andrew came to the same conclusion about duplicating MonitoredTask, the best 
idea is to try to reuse it :)
   
   The UI component is great and I agree that the "Tasks" page currently on the 
UI leaves something to be desired. I think your presentation of this data is 
excellent.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to