wirybeaver commented on PR #15203: URL: https://github.com/apache/pinot/pull/15203#issuecomment-2791404145
RCA: In my diff, I introduced a static singleton instance of the enableServerInstance map. This approach doesn’t cause issues in regular production environments since there’s typically only one pinot-broker per JVM. However, the IntegrationTest environment spins up multiple pinot-brokers within the same JVM process, which leads to all of them sharing and concurrently updating the singleton map. Prod Env JVM: single pinot-broker. BrokerRoutingManager (single writer), BalanceInstanceSelector(multi reader) Integration test JVM: 4 parallel test -> 4 pinot-brokers share the same static singleton, leading to the write conflicts. To fix this, I’ll remove the static singleton and instead pass the map explicitly where needed, ensuring each broker has its own isolated instance. -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
