[GitHub] Jackie-Jiang opened a new pull request #3898: In ClusterChangeMediator, remove sleep and make it notify based

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

Jackie-Jiang opened a new pull request #3898: In ClusterChangeMediator, remove sleep and make it notify based
URL: https://github.com/apache/incubator-pinot/pull/3898
 
 
   When we get the Helix cluster change callbacks, we want to process them ASAP
   Replace sleep() with wait() and notify() so that the new changes are processed immediately
   
   Testing done:
   - Check that the queue time matches with the processing time
   - Run tests without starting the cluster change handling thread
   - Reduce proactive change check interval to 1 second and check it interacts correctly with Helix callbacks

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org