-
Notifications
You must be signed in to change notification settings - Fork 56
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
Unregistration of KRaft nodes #128
Conversation
Signed-off-by: Jakub Scholz <www@scholzj.com>
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. Just a few nits.
Signed-off-by: Jakub Scholz <www@scholzj.com>
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.
Looks good. Proposal looks simple and effective until unregistration is supported in Kafka
Co-authored-by: PaulRMellor <47596553+PaulRMellor@users.noreply.github.com> Signed-off-by: Jakub Scholz <www@scholzj.com>
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, thanks :)
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.
The workaround makes sense to me. Thanks.
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
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
This proposal now seems to have 4 binding and 3 non-binding +1s. If there are no new comments raised until tomorrow CEST EOB, I will close it as approved. |
Signed-off-by: Jakub Scholz <www@scholzj.com>
Approved with 4 binding and 3 non-binding +1 votes. |
* Unregistration of KRaft nodes Signed-off-by: Jakub Scholz <www@scholzj.com> * Review comments PP Signed-off-by: Jakub Scholz <www@scholzj.com> * Apply suggestions from code review Co-authored-by: PaulRMellor <47596553+PaulRMellor@users.noreply.github.com> Signed-off-by: Jakub Scholz <www@scholzj.com> * Update index Signed-off-by: Jakub Scholz <www@scholzj.com> --------- Signed-off-by: Jakub Scholz <www@scholzj.com> Co-authored-by: PaulRMellor <47596553+PaulRMellor@users.noreply.github.com> Signed-off-by: Aswin A <aswin6303@gmail.com>
This proposal describes how we could handle the node unregistration which is part of the existing KRaft limitations.