jihoonson commented on issue #7285: Fix concurrency of VersionedIntervalTimeline
URL: 
https://github.com/apache/incubator-druid/issues/7285#issuecomment-512915757
 
 
   @leventov thanks for finding this bug. 
   
   > `BrokerServerView.getTimeline()` is another manifestation of this problem: 
it's an example of 
https://github.com/code-review-checklists/java-concurrency#concurrent-mutation-race.
   
   Why is this an example of concurrent mutation race rather than unsafe 
concurrent iteration? Looks like all users of `BrokerServerView.getTimeline()` 
just call `lookup()` on it.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to