jiajunwang commented on issue #363: Fix the race condition while Helix refresh cluster status cache. URL: https://github.com/apache/helix/pull/363#issuecomment-516551473 Thanks for reviewing. About the ConcurrentHashMap, I agree this becomes an overkill with this change. But the next move to get rid of it is not trivial. Better to do it once we restart the cache refactor work. As I commented in this thread, my idea is moving these flag, state info into the cache objects instead of the data provider. In this way, we won't have this map at all.
---------------------------------------------------------------- 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 With regards, Apache Git Services