Skip to content

Fix consumer stuck when disconnect with broker#18978

Open
pengxiangrui127 wants to merge 2 commits intoapache:branch-2.11from
pengxiangrui127:branch-2.11
Open

Fix consumer stuck when disconnect with broker#18978
pengxiangrui127 wants to merge 2 commits intoapache:branch-2.11from
pengxiangrui127:branch-2.11

Conversation

@pengxiangrui127
Copy link
Contributor

@pengxiangrui127 pengxiangrui127 commented Dec 19, 2022

Fixes #xyz

Master Issue: #xyz

Motivation

Modifications

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added integration tests for end-to-end deployment with large payloads (10MB)
  • Extended integration test for recovery after broker failure

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

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

[improve][client] Update MultiTopicsConsumerImpl.java
@github-actions
Copy link

@pengxiangrui127 Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

@github-actions github-actions bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Dec 19, 2022
@pengxiangrui127 pengxiangrui127 changed the title Branch 2.11 Fix consumer stuck when disconnect with broker Dec 19, 2022
@codelipenghui
Copy link
Contributor

@pengxiangrui127 Could you please provide more details about the BUG? and how your PR can fix the BUG?
This will help reviewers to understand the issue.

@github-actions
Copy link

The pr had no activity for 30 days, mark with Stale label.

@github-actions github-actions bot added the Stale label Jan 19, 2023
@Technoboy- Technoboy- added this to the 3.3.0 milestone Dec 22, 2023
@coderzc coderzc modified the milestones: 3.3.0, 3.4.0 May 8, 2024
@lhotari lhotari modified the milestones: 4.0.0, 4.1.0 Oct 14, 2024
@coderzc coderzc modified the milestones: 4.1.0, 4.2.0 Sep 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants