Skip to content
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

Fix ConcurrentOpenLongPairRangeSet remove all ranges #10656

Merged
merged 1 commit into from
May 21, 2021

Conversation

315157973
Copy link
Contributor

Motivation

Expected result: only the data in the current range is removed
Actual result: removed data for all ranges

@lhotari
Copy link
Member

lhotari commented May 20, 2021

@315157973 good catch! What was the impact of this bug?

@315157973
Copy link
Contributor Author

impact

Messages that have been acked may be repeatedly consumed

@lhotari lhotari added the type/bug The PR fixed a bug or issue reported a bug label May 20, 2021
@lhotari lhotari added this to the 2.8.0 milestone May 20, 2021
@merlimat merlimat merged commit 72777b0 into apache:master May 21, 2021
@merlimat merlimat added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label May 21, 2021
eolivelli pushed a commit to datastax/pulsar that referenced this pull request May 21, 2021
yangl pushed a commit to yangl/pulsar that referenced this pull request Jun 23, 2021
@315157973 315157973 deleted the remove branch July 19, 2021 11:11
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked/branch-2.7 Archived: 2.7 is end of life release/2.7.3 type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants