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

Change retry.backoff.* to apply to both producer and consumer #4561

Merged
merged 6 commits into from
May 8, 2024

Conversation

milindl
Copy link
Contributor

@milindl milindl commented Dec 14, 2023

After KIP-580, this property is used for both consumer and producer related operations, sort of as a general backoff.

The configuration needs to be updated to reflect this, else we generate a warning at 5and higher levels of logging.

@milindl milindl requested a review from a team May 8, 2024 06:13
Copy link
Contributor

@emasab emasab left a comment

Choose a reason for hiding this comment

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

LGTM

@milindl milindl merged commit 2587cac into master May 8, 2024
2 checks passed
@milindl milindl deleted the dev_property_nature_update branch May 8, 2024 10:11
anchitj pushed a commit that referenced this pull request Jun 10, 2024
* Change retry.backoff.* to apply to both producer and consumer

* Style fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants