Aravind-Suresh opened a new pull request, #11227: URL: https://github.com/apache/pinot/pull/11227
Fixes #11207. **Approach**: Starting a new thread inside the constructor escapes "this" object and results in data races as the object is not completely initialized while it gets shared with another thread. So, moved this logic to another function that gets invoked immediately after the constructor in all the usages (there's only one usage in the entire code + one no-op usage in test). **Test plan**: * Existing unit tests pass. * (TODO) e2e verification by running a few queries locally. -- 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
