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

Remove start-up dependency from ingress on partition processors #2206

Closed
tillrohrmann opened this issue Nov 4, 2024 · 0 comments · Fixed by #2207
Closed

Remove start-up dependency from ingress on partition processors #2206

tillrohrmann opened this issue Nov 4, 2024 · 0 comments · Fixed by #2207
Assignees
@tillrohrmann

Comments

@tillrohrmann
Copy link
Contributor

Because of #1651, we introduced #1684 as a temporary workaround. With #1651 being fixed now, we can remove the temporary solution which introduced a start-up dependency on the ingress which needed all partitions having an actively running leader.
@tillrohrmann tillrohrmann self-assigned this Nov 4, 2024
tillrohrmann added a commit to tillrohrmann/restate that referenced this issue Nov 4, 2024
@tillrohrmann
Remove temporary solution for avoiding stuck invocations because of l…
17f6c79 
…ost SubmitNotifications

With restatedev#1651 being fixed, we no longer need to wait for all partitions to have a
running leader before we can start the ingress to avoid stucks invocations. That's
because the ingress will now retry until it finds an active leader for a given partition.

This fixes restatedev#2206.
@tillrohrmann tillrohrmann mentioned this issue Nov 4, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tillrohrmann tillrohrmann

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@tillrohrmann