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

[branch-3.1][improve][broker] backlog quota exceed limit log replaced with debug #22475

Merged

Conversation

mukesh154
Copy link
Contributor

Fixes #22474

Motivation

Currently, when backlog quota limit exceeded and producers still send messages, it generates lot of logs in short period of time. The logs are logged with info. We can use debug instead of info.

Modifications

  • Replaced the info with debug for logs when the backlog quota limit exceeded.

Verifying this change

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

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

  • 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

Check the box below or label this PR directly.

Need to update docs?

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

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Apr 10, 2024
@nicoloboschi nicoloboschi added this to the 3.3.0 milestone Apr 12, 2024
@nicoloboschi nicoloboschi merged commit 767f0b2 into apache:branch-3.1 Apr 12, 2024
18 of 20 checks passed
@nicoloboschi nicoloboschi changed the title [improve][broker] backlog quota exceed limit log replaced with debug [branch-3.1][improve][broker] backlog quota exceed limit log replaced with debug Apr 12, 2024
@nicoloboschi nicoloboschi removed this from the 3.3.0 milestone Apr 12, 2024
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 16, 2024
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 17, 2024
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 22, 2024
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants