Skip to content

Trait definitions for atomic write transactions - #755

Merged
adzialocha merged 5 commits into
mainfrom
atomic-transactions
Jul 14, 2025
Merged

Trait definitions for atomic write transactions#755
adzialocha merged 5 commits into
mainfrom
atomic-transactions

Conversation

@adzialocha

@adzialocha adzialocha commented May 27, 2025

Copy link
Copy Markdown
Member

This PR introduces trait definitions for expressing atomic write transactions. This will be required to separate store methods into read and write queries and allow crash-resiliant write transactions across the whole p2panda stack.

Next Steps:

  • Actually remove "write" methods from OperationStore and LogStore
  • Implement new WriteToStore trait on Operation
  • Decide where store trait definitons and implementations like OperationStore, LogStore, later probably EncryptionStore and AuthStore, etc. should live (either in p2panda-store or in their regarding crates)

📋 Checklist

  • Add tests that cover your changes
  • Add this PR to the Unreleased section in CHANGELOG.md
  • New files contain a SPDX license header

@adzialocha
adzialocha marked this pull request as ready for review May 27, 2025 14:14
@adzialocha
adzialocha requested review from mycognosist and sandreae and removed request for sandreae May 27, 2025 14:14

@sandreae sandreae left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

All looks very good, excited to start working with these new interfaces :-)

@adzialocha
adzialocha merged commit 297f0cc into main Jul 14, 2025
@adzialocha
adzialocha deleted the atomic-transactions branch October 1, 2025 16:16
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.

2 participants