-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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-8399: bring back internal.leave.group.on.close config for KStream #6779
Conversation
3bd6cc6
to
3be5776
Compare
Retest this please |
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.
From what I can tell, LGTM.
Can we run the system tests (at least |
@abbccdda Made a pass, looks good overall. Could you also add back the unit test checking that upon shutting down Streams does not send leave group request as well, so that we can capture this earlier in unit tests than system tests? |
Second try passed too: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2616/ |
retest this please |
Merged #6779 into trunk |
Cherry-picked to 2.3 as well. |
…am (apache#6779) As title states. We plan to merge this to both trunk and 2.3 if it could fix the stream system tests globally. Reference implementation: apache#6673 Reviewers: Guozhang Wang <wangguoz@gmail.com>, Matthias J. Sax <mjsax@apache.org>
As title states. We plan to merge this to both trunk and 2.3 if it could fix the stream system tests globally.
Reference implementation: #6673
Committer Checklist (excluded from commit message)