ServerSelector: Synchronize getAllServers().

apache / druid

Apache Druid: a high performance real-time analytics database.

https://druid.apache.org/

Apache License 2.0

13.53k stars 3.71k forks source link

ServerSelector: Synchronize getAllServers(). #17499

Closed gianm closed 18 hours ago

gianm commented 6 days ago

This method was missing some required synchronization. This patch also adds GuardedBy annotations to historicalServers and realtimeServers, which would have caught it.