-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[event-hubs] Introduce a random delay when starting the event processor loop #7111
[event-hubs] Introduce a random delay when starting the event processor loop #7111
Conversation
…id potential contention if event processors start near the same time.
/azp run js - eventhub - ci |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run js - eventhub - ci |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run js - eventhubs-client - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run js - eventhubs-client - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
@chradek, I believe the issue this PR is meant to fix is in the current milestone. Are the changes here still relevant or has anything changed since this PR was created? @richardpark-msft, can you please update the PR to resolve the merge conflicts? |
I might hold off on this until I've gotten the PR for load balancing changes out in case some of how we initialize things changes. .NET is also doing something interesting where they only claim one partition at a time in the loop to prevent these sort of problems, and I'd like to see if we can incorporate some of that in our balanced load balancer strategy. |
It really seems like the need for this doesn't exist. Closing for now, we can always reopen it if we change our minds. |
Add in a randomized delay when starting the event processor loop for the first time, to avoid processors running on the exact same cadence and getting into a deadlock.
Fixes #7017