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

Added reconnect_backoff_max_ms option to set maximum reconnection time... #308

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arenard
Copy link

@arenard arenard commented Feb 24, 2019

...to exponential reconnection backoff

Set default value to reconnect_backoff_ms disable exponential reconnection backoff strategy implemented in kafka client. See : https://github.com/apache/kafka/blob/9a0ea25fee85837748145d37c69cf4d9bb7f9933/clients/src/main/java/org/apache/kafka/clients/CommonClientConfigs.java#L114

…ime...

to exponential reconnection backoff
@arenard arenard force-pushed the feat-kafka-max-reconnect-backoff branch from 3deebe5 to 4c3042a Compare February 24, 2019 17:16
@arenard
Copy link
Author

arenard commented Feb 24, 2019

Seen it after but I moved reconnect_backoff_ms option datatype from string to number for coherence with reconnect_backoff_max_ms option.

To avoid two major version changes, it may be better to merge this after: #309 because it correct all misleading options datatypes.

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.

1 participant