s0nskar commented on PR #2819:
URL: https://github.com/apache/celeborn/pull/2819#issuecomment-4234266494

   @leixm have we seen any issues because of this in production setup or is 
this precaution? 
   
   I am asking this because this change cause split brain till new Master 
receives heartbeats from all workers (around 30-60s) and quota might be 
inconsistent during this time. 
   - It will allow new applications to admit even though quota is breached.
   - Interrupt of high shuffle applications will be delayed.


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

Reply via email to