Skip to content

fix(cli): integrate checkpoint resume - #1250

Open
Jaswanth-Sriram-Veturi wants to merge 1 commit into
TauricResearch:mainfrom
Jaswanth-Sriram-Veturi:fix/cli-checkpoint-resume
Open

fix(cli): integrate checkpoint resume#1250
Jaswanth-Sriram-Veturi wants to merge 1 commit into
TauricResearch:mainfrom
Jaswanth-Sriram-Veturi:fix/cli-checkpoint-resume

Conversation

@Jaswanth-Sriram-Veturi

Copy link
Copy Markdown

Fixes #1249

Problem

The CLI accepted --checkpoint, but run_analysis() directly streamed a graph compiled without a checkpointer or stable
thread_id. Crashed CLI runs therefore could not resume.

Changes

  • Configure the SQLite checkpointer on the CLI streaming path.
  • Propagate the deterministic checkpoint thread_id.
  • Pass None when resuming so completed nodes are not replayed.
  • Clear checkpoints after successful completion.
  • Close the checkpointer in finally.
  • Expose checkpoint lifecycle methods as public graph methods because the CLI uses them.
  • Add a regression test that asserts completed nodes execute only once across crash and resume.

Attach the SQLite checkpointer and stable thread IDs to CLI runs.
Resume from existing checkpoints without replaying completed nodes.
Clear checkpoints after successful completion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI --checkpoint flag does not actually enable checkpoint resume

1 participant