saintstack commented on pull request #1826: URL: https://github.com/apache/hbase/pull/1826#issuecomment-649243696
Updating TaskMonitor inside the method that deserializes procedure state data is a total surprise. I thought TaskMonitor best effort rather than a true view especially as so many systems are delinquent regards keeping up TaskMonitor state. On the current extraordinary effort to keep it in-line, I'd say, nice, but it is causing bigger issues so bypass for this rare condition? On the flag juggling around resume, lets do that in another issue. Let this bug fix through? I for one do not look at TaskMonitor figuring state of Procedures. Do others? Thanks. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
