zaynt4606 commented on PR #3828:
URL: https://github.com/apache/celeborn/pull/3828#issuecomment-6360018198

   @cxzl25 I pushed 56b10da0a to address the remaining timeout issue. The 
timeout is now applied under the status transition lock only when a 
decommission request is accepted, so ignored duplicate or concurrent requests 
cannot overwrite it. The drain loop and shutdown hook use the same accepted 
timeout.
   
   I also moved status manager initialization before the HTTP server and 
heartbeats start, so it is ready when exit requests arrive.
   
   Could you take a look and let me know whether this approach works for you?


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