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

Meta reload might happen after the partition processor starts processing enqueued messages #361

Closed
Tracked by #40
slinkydeveloper opened this issue May 4, 2023 · 1 comment · Fixed by #365
Closed
Tracked by #40

Meta reload might happen after the partition processor starts processing enqueued messages #361

slinkydeveloper opened this issue May 4, 2023 · 1 comment · Fixed by #365
Assignees
@slinkydeveloper
Labels
bug Something isn't working
Milestone
1A

Comments

@slinkydeveloper
Copy link
Contributor

The partition processor might have some enqueued work, and might start processing it before meta reloads the meta information from disk.
@slinkydeveloper slinkydeveloper added the bug Something isn't working label May 4, 2023
@slinkydeveloper slinkydeveloper added this to the 1A milestone May 4, 2023
@slinkydeveloper slinkydeveloper self-assigned this May 4, 2023
@slinkydeveloper slinkydeveloper mentioned this issue May 4, 2023
Closed
@slinkydeveloper
Copy link
Contributor Author

A proper fix is described here: #91. For M1A we could simplify the solution by simply enforcing the ordering of components to reload on restarts.

@slinkydeveloper slinkydeveloper mentioned this issue May 5, 2023
Merged
slinkydeveloper added a commit that referenced this issue May 5, 2023
@slinkydeveloper
Let meta reload before worker starts (#365)
53e0f52 
* Fix #361

* Add a debug assert to check if the service started
@tillrohrmann tillrohrmann mentioned this issue May 5, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slinkydeveloper slinkydeveloper

Labels
bug Something isn't working
Projects
None yet
Milestone
1A
Development

Successfully merging a pull request may close this issue.

Let meta reload before worker starts slinkydeveloper/restate-old
1 participant
@slinkydeveloper