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 2019/07/18 17:38:56 UTC
[GitHub] [incubator-druid] jihoonson commented on issue #7285: Fix
concurrency of VersionedIntervalTimeline
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:
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