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

[Broker] Change broker producer fence log level #14196

Conversation

Demogorgon314
Copy link
Member

Fixes #12290

Motivation

Fixes #12290

Modifications

  • Change broker producer fence log level to debug level.
  • Change others producer exception log level to warn level.

Documentation

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 the doc-not-needed Your PR changes do not impact docs label Feb 10, 2022
mattisonchao
mattisonchao previously approved these changes Feb 10, 2022
@Demogorgon314
Copy link
Member Author

/pulsarbot run-failure-checks

1 similar comment
@Demogorgon314
Copy link
Member Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui modified the milestones: 2.10.0, 2.11.0 Feb 14, 2022
@Demogorgon314
Copy link
Member Author

/pulsarbot run-failure-checks

gaozhangmin
gaozhangmin previously approved these changes Feb 15, 2022
jiazhai
jiazhai previously approved these changes Feb 15, 2022
Technoboy-
Technoboy- previously approved these changes Feb 15, 2022
Copy link
Contributor

@Technoboy- Technoboy- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please help check the comment.

@Demogorgon314 Demogorgon314 force-pushed the fix/change-broker-produce-fence-log-level branch from f2ca5d6 to 22e000d Compare February 15, 2022 06:54
@codelipenghui codelipenghui modified the milestones: 2.11.0, 2.10.0 Feb 16, 2022
@codelipenghui codelipenghui merged commit 141ea9b into apache:master Feb 16, 2022
@Demogorgon314 Demogorgon314 deleted the fix/change-broker-produce-fence-log-level branch February 17, 2022 05:36
michaeljmarshall pushed a commit that referenced this pull request Feb 23, 2022
@michaeljmarshall michaeljmarshall added the cherry-picked/branch-2.8 Archived: 2.8 is end of life label Feb 23, 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
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/broker cherry-picked/branch-2.8 Archived: 2.8 is end of life cherry-picked/branch-2.9 Archived: 2.9 is end of life doc-not-needed Your PR changes do not impact docs release/2.8.3 release/2.9.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Too many Error messages in the broker
10 participants