KAFKA-13081: detect doubly assigned parition (for v2.8)#11068
Merged
ableegoldman merged 1 commit intoapache:2.8from Aug 19, 2021
Merged
KAFKA-13081: detect doubly assigned parition (for v2.8)#11068ableegoldman merged 1 commit intoapache:2.8from
ableegoldman merged 1 commit intoapache:2.8from
Conversation
showuon
changed the title
Member
Author
|
@ableegoldman , please take a look. Thank you. Failed tests are unrelated. |
hachikuji
pushed a commit
that referenced
this pull request
Oct 19, 2021
…ive assignor (#11068) This is the fix 1 and fix 2 in #10985 for v2.7, including the tests. Uses the generation to invalidate previous assignments that claim partitions but no longer own them, and implements an additional safety net to handle any case in which doubly-claimed partitions slip in to the input anyway Reviewers: Anna Sophie Blee-Goldman <ableegoldman@apache.org>
hachikuji
pushed a commit
that referenced
this pull request
Oct 19, 2021
…ive assignor (#11068) This is the fix 1 and fix 2 in #10985 for v2.6, including the tests. Uses the generation to invalidate previous assignments that claim partitions but no longer own them, and implements an additional safety net to handle any case in which doubly-claimed partitions slip in to the input anyway Reviewers: Anna Sophie Blee-Goldman <ableegoldman@apache.org>
hachikuji
pushed a commit
that referenced
this pull request
Oct 19, 2021
…ive assignor (#11068) This is the fix 1 and fix 2 in #10985 for v2.6, including the tests. Uses the generation to invalidate previous assignments that claim partitions but no longer own them, and implements an additional safety net to handle any case in which doubly-claimed partitions slip in to the input anyway Reviewers: Anna Sophie Blee-Goldman <ableegoldman@apache.org>
andrewegel
pushed a commit
to confluentinc/kafka
that referenced
this pull request
Oct 22, 2021
…ive assignor (apache#11068) This is the fix 1 and fix 2 in apache#10985 for v2.6, including the tests. Uses the generation to invalidate previous assignments that claim partitions but no longer own them, and implements an additional safety net to handle any case in which doubly-claimed partitions slip in to the input anyway Reviewers: Anna Sophie Blee-Goldman <ableegoldman@apache.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is the fix 1 and fix 2 in #10985 for v2.8, including the tests.
Committer Checklist (excluded from commit message)