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

Reduce the default for max_kafka_throttle_delay_ms #9935

Merged

Conversation

dlex
Copy link
Contributor

@dlex dlex commented Apr 10, 2023

max_kafka_throttle_delay_ms default value changed to 30s to align with the default setting of session.timeout.ms in java client and in librdkafka which is 45s there. By setting the default max throttle delay lower than session timeout, it is possible to avoid heartbeat timeouts for librdkafka clients (librdkafka does not support KIP-219 as of now and therefore relies on broker doing throttling) with default configuration on both client and broker.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.1.x
  • v22.3.x
  • v22.2.x

Release Notes

Improvements

  • Maximum throttle time for Kafka API requests reduced to 30s to avoid heartbeat timeouts in default client configurations.

`max_kafka_throttle_delay_ms` default value changed to 30s to align
with the default setting of `session.timeout.ms` in java client and in
librdkafka which is 45s there. By setting the default max throttle delay
lower than session timeout, it is possible to avoid heartbeat timeouts
for librdkafka clients (librdkafka does not support KIP-219 as of now
and therefore relies on broker doing throttling) with default
configuration on both client and broker.
@dlex dlex marked this pull request as ready for review April 10, 2023 22:24
@dlex dlex requested a review from graphcareful April 10, 2023 22:24
@piyushredpanda piyushredpanda merged commit a222e2a into redpanda-data:dev Apr 12, 2023
@piyushredpanda
Copy link
Contributor

/backport v23.1.x

@dlex dlex deleted the lower-max-kafka-throttle-delay branch April 13, 2023 01:56
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.

3 participants