kfaraz commented on PR #18824: URL: https://github.com/apache/druid/pull/18824#issuecomment-3625988366
Thanks for the clarification, @jtuglu1 ! It makes sense to have the inventory view be consistent with the peons. I also encountered another possible race condition related `segmentsMarkedToDrop`. In `ServerHolder.initializeQueuedSegments()`, we make a call to `peon.getSegmentsInQueue()` and `peon.getSegmentsMarkedToDrop()` in quick succession, but it is possible that between these calls, the state of the peon changes. I have tried to address this race condition and the reconciliation with the server inventory snapshot in the latest commit. Let me know if it makes sense and addresses your issue. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
