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

[Flaky-Test] BacklogQuotaManagerTest#testProducerExceptionAndThenUnblockSizeQuota #14213

Conversation

gaozhangmin
Copy link
Contributor

@gaozhangmin gaozhangmin commented Feb 10, 2022

Fixes #14203

Motivation

sendTimeout error is caused by there were pendingMessages which created by the previous quota exceed failed send.

Modifications

Recreate produce to clean pending messages.

Documentation

Check the box below or label this PR directly (if you have committer privilege).

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@github-actions github-actions bot added doc-not-needed Your PR changes do not impact docs and removed doc-not-needed Your PR changes do not impact docs labels Feb 10, 2022
@github-actions
Copy link

@gaozhangmin:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions
Copy link

@gaozhangmin:Thanks for providing doc info!

@github-actions github-actions bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Feb 10, 2022
@gaozhangmin
Copy link
Contributor Author

/pulsarbot run-failure-checks

@gaozhangmin
Copy link
Contributor Author

@lhotari PTAL

@gaozhangmin
Copy link
Contributor Author

@nicoloboschi @merlimat PTAL

@gaozhangmin
Copy link
Contributor Author

@codelipenghui PTAL

@codelipenghui codelipenghui added this to the 2.11.0 milestone Feb 16, 2022
@codelipenghui codelipenghui merged commit cd172e7 into apache:master Feb 16, 2022
@codelipenghui codelipenghui modified the milestones: 2.11.0, 2.10.0 Feb 16, 2022
gaoran10 pushed a commit that referenced this pull request Mar 1, 2022
@gaoran10 gaoran10 added the cherry-picked/branch-2.9 Archived: 2.9 is end of life label Mar 2, 2022
nicoloboschi pushed a commit to datastax/pulsar that referenced this pull request Mar 2, 2022
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/test cherry-picked/branch-2.9 Archived: 2.9 is end of life doc-not-needed Your PR changes do not impact docs release/2.9.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flaky-test: BacklogQuotaManagerTest.testProducerExceptionAndThenUnblockSizeQuota
3 participants