Skip to content

perf(api): reuse transaction checkpoint across staged statements - #1834

Merged
DecisionNerd merged 4 commits into
mainfrom
perf/1807-remove-per-statement-workspace-backups
Oct 5, 2026
Merged

DecisionNerd merged 4 commits into
mainfrom
perf/1807-remove-per-statement-workspace-backups

Conversation

@DecisionNerd

@DecisionNerd DecisionNerd commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Capture one rollback checkpoint at the owning transaction boundary and reuse it across staged Cypher statements.
  • Let inner statement lifecycles defer rollback to the transaction owner while preserving successful statement refreshes and read-your-writes behavior.
  • Add real facade work counters for checkpoint captures, source and copied bytes, files, and flushes, plus a later-statement failure/reopen regression.

Validation


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository: CurateLabs/graphforge/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8c3d5908-ef84-4d4b-9a9e-98146c09f397

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the core Core source code changes label Oct 5, 2026
@DecisionNerd
DecisionNerd force-pushed the perf/1807-remove-per-statement-workspace-backups branch from eb7c532 to 6b11d8a Compare October 5, 2026 06:58
@DecisionNerd
DecisionNerd added this pull request to the merge queue Oct 5, 2026
Merged via the queue into main with commit a53ca88 Oct 5, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core source code changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

perf(api): remove per-statement whole-workspace transaction backups

1 participant