rondagostino commented on pull request #10991: URL: https://github.com/apache/kafka/pull/10991#issuecomment-876040323
Just realized that one potential problem with the log message approach is that the last broker in a 3-broker cluster with 3 co-located controllers dos not shut down cleanly. The current approach deals with this by sending SIGKILL instead of SIGTERM. The log message approach doesn't deal with it, so the shutdown in the system test would end up timing out after 60 seconds and then sending SIGKILL later as part of the cleanup (the test would still pass). Not sure if this adds credence to the current approach? -- 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]
