Skip to content

[OLD] Restart invocation #3360

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
Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

[OLD] Restart invocation #3360

wants to merge 13 commits into from

Conversation

slinkydeveloper
Copy link
Contributor

@slinkydeveloper slinkydeveloper commented Jun 5, 2025 •
edited
Loading

Fix #3347, depends on #3227 and #3362

@slinkydeveloper slinkydeveloper force-pushed the issues/3347 branch 3 times, most recently from c597ffc to 474b66f Compare June 5, 2025 11:36
@slinkydeveloper slinkydeveloper mentioned this pull request Jun 5, 2025
Closed
@slinkydeveloper slinkydeveloper added the release-blocker Blocker for the next release label Jun 5, 2025
@slinkydeveloper slinkydeveloper mentioned this pull request Jun 5, 2025
Open
@slinkydeveloper
Copy link
Contributor Author

E2E test here restatedev/e2e#379

This was referenced Jun 5, 2025
Merged
Open
@slinkydeveloper
New APIs in the InvocationStatusTable and JournalTable to accommodate…
3f9b912 
… storing "archived" journals and statuses. These APIs will be used by the following commits.
@slinkydeveloper
Expose invocation epoch in SQL
04c217f
@slinkydeveloper
Interface for the restart feature, plus add functionality in purge in…
0608d22 
…vocation/purge journal to selectively remove an invocation epoch
@slinkydeveloper
Implementation of the Restart feature
5c6855b
@slinkydeveloper
Adapt the purge commands to understand invocation epoch. We also add …
3a01af2 
…an invariant that when you remove the last epoch, it removes all the old attempts too. This simplifies a bit the behavior and how we want to show it.
@slinkydeveloper
Make sure the cleaner now deals with the epoch + fix bug
f5a1fe1
@slinkydeveloper
New admin REST API for the restart
c5077f8
@slinkydeveloper
Implement the new storage APIs
91441a2
@slinkydeveloper
Make sure we read from journal table v2, even if the pinned deploymen…
a110343 
…t is not set. This is a new condition that can happen with reset, because we don't re-migrate the journal table back to v1.
@slinkydeveloper
Mindless plumbing
0f22f0c
@slinkydeveloper
Return the new invocation epoch in restart_invocation API
9925638
@slinkydeveloper
More thorough test
12b8c3a
@slinkydeveloper slinkydeveloper mentioned this pull request Jun 11, 2025
Merged
@slinkydeveloper slinkydeveloper removed the release-blocker Blocker for the next release label Jun 11, 2025
@slinkydeveloper slinkydeveloper changed the title Restart invocation [OLD] Restart invocation Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Restart invocation
1 participant
@slinkydeveloper