[GitHub] [hadoop-ozone] elek commented on issue #861: HDDS-3483. Handle unhealthy replica state of an open container.

[GitBox <gi...@apache.org>]

elek commented on issue #861:
URL: https://github.com/apache/hadoop-ozone/pull/861#issuecomment-618497085


   Thanks to open this issue @nandakumar131, one minor comment:
   
   {code}
   +    // Wait for EventQueue to call the event handler
   +    Thread.sleep(100L);
   +    Mockito.verify(closeContainerHandler, Mockito.times(1))
   +        .onMessage(id, eventQueue);
   {code}
   
   FYI: Instead of using `Thread.sleep` we have a better api: `eventQueue.processAll(5000);`
   
   (Sorry, I was not fast enough to comment it during the 1-2 hours to window while it was open. Keeping it open more time can give everybody a chance to check it (especially as US is waking up just now))  
   
   


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org