gharris1727 opened a new pull request, #12830:
URL: https://github.com/apache/kafka/pull/12830

   Signed-off-by: Greg Harris <greg.har...@aiven.io>
   
   The GRACEFUL_SHUTDOWN_TIMEOUT_MS for the Trogdor JsonRestServer is 100ms. In 
heavily loaded CI environments, this timeout can be exceeded. When this 
happens, it causes the jettyServer.stop() and jettyServer.destroy() calls to 
throw exceptions, which prevents shutdownExecutor.shutdown() from running. This 
has the effect of causing the JsonRestServer::waitForShutdown method to block 
for 1 day, which exceeds the 120s timeout on the CoordinatorTest (and any other 
test relying on MiniTrogdorCluster).
   
   This change makes it such that the graceful shutdown timeout is less likely 
to be exceeded, and when it is, the timeout does not cause the waitForShutdown 
method to block for much longer than the graceful shutdown timeout.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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: jira-unsubscr...@kafka.apache.org

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

Reply via email to