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.

Sign up for GitHub

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

Jump to bottom

Fix IndexOutOfBoundsException in the KeyShared subscription when dispatching messages to consumers #8024

Merged
merged 1 commit into from Sep 11, 2020
Merged

Fix IndexOutOfBoundsException in the KeyShared subscription when dispatching messages to consumers #8024

merged 1 commit into from Sep 11, 2020

Conversation

codelipenghui
Copy link
Contributor

Fixes #7941

Motivation

Fix IndexOutOfBoundsException in the KeyShared subscription when dispatching messages to consumers

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

Documentation

  • Does this pull request introduce a new feature? (no)

@codelipenghui codelipenghui self-assigned this Sep 10, 2020
@codelipenghui codelipenghui added this to the 2.7.0 milestone Sep 10, 2020
@codelipenghui codelipenghui added the type/bug The PR fixed a bug or issue reported a bug label Sep 10, 2020
@codelipenghui
Copy link
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit f9988b4 into apache:master Sep 11, 2020
@codelipenghui codelipenghui deleted the penghui/fix-7941 branch September 11, 2020 08:22
codelipenghui added a commit to streamnative/pulsar-archived that referenced this pull request Sep 11, 2020
@codelipenghui
Avoid consumer stuck when available permits is a negative value. (apa…
880c485 
…che#8024)

Fixes apache#7941

### Motivation

Fix IndexOutOfBoundsException in the KeyShared subscription when dispatching messages to consumers

(cherry picked from commit f9988b4)
@alexku7 alexku7 mentioned this pull request Sep 15, 2020
Closed
@sijie sijie mentioned this pull request Sep 17, 2020
Closed
@alexku7 alexku7 mentioned this pull request Sep 21, 2020
Closed
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Oct 3, 2020
@codelipenghui
Avoid consumer stuck when available permits is a negative value. (apa…
a7c6078 
…che#8024)

Fixes apache#7941

### Motivation

Fix IndexOutOfBoundsException in the KeyShared subscription when dispatching messages to consumers
wolfstudy pushed a commit that referenced this pull request Oct 30, 2020
@codelipenghui @wolfstudy
Avoid consumer stuck when available permits is a negative value. (#8024)
807f550 
Fixes #7941

### Motivation

Fix IndexOutOfBoundsException in the KeyShared subscription when dispatching messages to consumers

(cherry picked from commit f9988b4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sijie sijie sijie approved these changes

@jiazhai jiazhai jiazhai approved these changes

@merlimat merlimat Awaiting requested review from merlimat

Assignees

@codelipenghui codelipenghui

Labels
release/2.6.2 type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Milestone
2.7.0
Development

Successfully merging this pull request may close these issues.

Consumer can't get messages with earliest position subscription type under some circumstances
3 participants
@codelipenghui @sijie @jiazhai