-
Notifications
You must be signed in to change notification settings - Fork 13.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KAFKA-14706 Move/rewrite ShutdownableThread to server-common module. #13234
Conversation
server-common/src/main/java/org/apache/kafka/server/util/ShutdownableThread.java
Outdated
Show resolved
Hide resolved
21d43b5
to
a020dcd
Compare
Test failure does not seem to be related to the changes in the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
I won't have a chance to review for a few more days - if @showuon can do it sooner, let's not block on me. |
I'll take a look today. |
One failing test is not related to this change, ready to be merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had another look, lgtm!
KAFKA-14706 Move/rewrite ShutdownableThread to
server-common
module.Committer Checklist (excluded from commit message)