jadami10 commented on pull request #7961:
URL: https://github.com/apache/pinot/pull/7961#issuecomment-1004384141


   > There could be multiple servers running on the same machine (e.g. running 
integration test that spins up multiple servers). Will the cleanup from the 2nd 
server wipe out the `tmp-` directory of the 1st server?
   
   That's a fair point. The integration tests passed which makes me think 
they're ok. But if someone is actually running it this way in production, this 
would be bad for them. The directories themselves have no tie to the server, so 
maybe the only option is to provide a longer threshold. Either delete for than 
N hours ago or offer a configuration for it?


-- 
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

Reply via email to