You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2022/06/10 16:37:34 UTC

[GitHub] [kafka] divijvaidya opened a new pull request, #12283: MINOR: Shutdown jmx reporter using a non-daemon kafka thread

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

   Use the internal `Exit.addShutdownHook` to close jmx reporter since this internal version has some nicer logging.
   


-- 
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-unsubscribe@kafka.apache.org

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


[GitHub] [kafka] mimaison merged pull request #12283: MINOR: Shutdown jmx reporter using internal utility method

Posted by GitBox <gi...@apache.org>.
mimaison merged PR #12283:
URL: https://github.com/apache/kafka/pull/12283


-- 
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-unsubscribe@kafka.apache.org

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


[GitHub] [kafka] divijvaidya commented on pull request #12283: MINOR: Shutdown jmx reporter using internal utility method

Posted by GitBox <gi...@apache.org>.
divijvaidya commented on PR #12283:
URL: https://github.com/apache/kafka/pull/12283#issuecomment-1154035937

   @mimaison perhaps you could review this quick code change?


-- 
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-unsubscribe@kafka.apache.org

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