Skip to content

feat(sharedb): commit snapshots and ops atomically#9

Merged
edocevol merged 1 commit into
mainfrom
feat/sharedb-atomic-commit
Apr 28, 2026
Merged

feat(sharedb): commit snapshots and ops atomically#9
edocevol merged 1 commit into
mainfrom
feat/sharedb-atomic-commit

Conversation

@edocevol
Copy link
Copy Markdown
Owner

Summary

  • add Backend.CommitOp for atomic snapshot and op-log persistence
  • update Server.SubmitWithRequest to commit non-empty operations through CommitOp
  • implement atomic memory backend commits under one mutex
  • implement Redis CommitOp with Lua precondition checks before HSET/RPUSH
  • add regression test proving failed commits do not advance snapshots

Test Plan

  • go test ./...
  • cd sharedb && go test ./...
  • cd sharedb && go vet ./...
  • cd sharedb && go test -race ./...

Review

  • independent reviewer subagent: PASS

@edocevol edocevol merged commit 2bde4fc into main Apr 28, 2026
1 check passed
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.

1 participant