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

WIP - Checkpoint override to allow syncing from non-finalized state #9204

Draft
wants to merge 16 commits into
base: master
Choose a base branch
from
Draft

WIP - Checkpoint override to allow syncing from non-finalized state #9204

wants to merge 16 commits into from
+110 −63

Conversation

gfukushima
Copy link
Contributor

PR Description

Fixed Issue(s)

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

@gfukushima
add checkpoint override option
8362e43 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
fix tests
724b902 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
spotless
e970587 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
hide experimental option
a2f2612 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
set checkpoint using finalized checkpoint fromm state
e2c7386 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
Merge branch 'master' into checkpoint-override
c149a0c
@gfukushima gfukushima changed the title Checkpoint override to allow syncing from non-finalized state WIP - Checkpoint override to allow syncing from non-finalized state Mar 5, 2025
gfukushima added 10 commits March 5, 2025 11:58
@gfukushima
comment out anchor point validation that assumes it's always the same…
2a03e8e 
… as checkpoint

Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
spotless
30a7923 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
Merge remote-tracking branch 'origin/checkpoint-override' into checkp…
2d4e81e 
…oint-override
@gfukushima
massive waste of resources fetching 2 states I know, but it's just fo…
67c68fb 
…r the sake of spiking

Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
spotless
337cf57 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
just get the test passing for now
1f6e32b 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
skip another check
e926112 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
spotless
1585de8 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
errorprone
c5c1a01 
Signed-off-by: Gabriel Fukushima <[email protected]>
@gfukushima
spotless
32790ca 
Signed-off-by: Gabriel Fukushima <[email protected]>
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.
1 participant
@gfukushima