You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/01/30 07:25:14 UTC

[GitHub] [druid] leventov opened a new issue #9292: CoordinatorRuleManager.rules doesn't need to store ConcurrentHashMap

leventov opened a new issue #9292: CoordinatorRuleManager.rules doesn't need to store ConcurrentHashMap
URL: https://github.com/apache/druid/issues/9292
 
 
   `CoordinatorRuleManager` didn't have a race condition prior to #6898, actually, because the `ConcurrentHashMap` is never modified after assigning into `AtomicReference`. So the proper fix was (and the proper action now is) to replace ConcurrentHashMap with simple HashMap. 

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [druid] jihoonson closed issue #9292: CoordinatorRuleManager.rules doesn't need to store ConcurrentHashMap

Posted by GitBox <gi...@apache.org>.
jihoonson closed issue #9292: CoordinatorRuleManager.rules doesn't need to store ConcurrentHashMap
URL: https://github.com/apache/druid/issues/9292
 
 
   

----------------------------------------------------------------
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


With regards,
Apache Git Services

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