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

Allow for no retry jitter #547

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

Conversation

bblanke
Copy link

@bblanke bblanke commented Apr 27, 2021

There needs to be a way to eliminate retry jitter. For my use case, I need the autobahn client to attempt to reconnect to a server every 2 seconds. With the pseudo-random retry times introduced by the jitter, my retry time can end up anywhere from a few milliseconds to 30 seconds (depending on how long I leave it). That's not ideal for me.

The code already has a clause that only applies retry jitter if the _retry_delay_jitter property is truthy. So if the property were 0, then no retry jitter would be applied.

However, due to the way self._retry_delay_jitter is initialized, it will never accept a value of 0. This pull request changes that and allows self._retry_delay_jitter to be set to 0.

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.

None yet

1 participant