[GitHub] [pinot] Jackie-Jiang commented on pull request #10238: Fix race condition on removing aged deleted segments

["Jackie-Jiang (via GitHub)" <gi...@apache.org>]

Jackie-Jiang commented on PR #10238:
URL: https://github.com/apache/pinot/pull/10238#issuecomment-1421548249

   Does it run into exception or just logged some warning? I'd suggest not introducing the lead controller concept into this module, but simply handle the case of file deleted during the checks because lead controller change is not guaranteed to be visible to all controllers at the same time.


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org