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.

Sign up for GitHub

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

Jump to bottom

[EPH] update EventProcessor to use PartitionManager #4542

Closed
chradek opened this issue Jul 30, 2019 · 1 comment
Closed

[EPH] update EventProcessor to use PartitionManager #4542

chradek opened this issue Jul 30, 2019 · 1 comment
Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs
Milestone
Sprint 156

Comments

@chradek
Copy link
Contributor

chradek commented Jul 30, 2019

Depends on #4535 and #4538

This issue is to track updating the EventProcessor to use the PartitionManager to claim partition ownerships. Within the EventProcessor's loop, it should check which partitions it has ownership of. Any partitions it doesn't own it should attempt to claim before creating a new pump.

Load-balancing partitions across multiple EventProcessors does not need to be part of this PR.
@chradek chradek added Client This issue points to a problem in the data-plane of the library. Event Hubs labels Jul 30, 2019
@chradek chradek modified the milestone: Sprint 156 Jul 30, 2019
@ramya-rao-a ramya-rao-a added this to the Sprint 156 milestone Jul 31, 2019
@chradek
Copy link
Contributor Author

chradek commented Aug 6, 2019

Completed by #4583 and #4621

@chradek chradek closed this as completed Aug 6, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs
Projects
None yet
Milestone
Sprint 156
Development

No branches or pull requests
2 participants
@ramya-rao-a @chradek