Skip to content

Feature/store transaction#131

Merged
zonotope merged 13 commits intomainfrom
feature/store-transaction
Apr 16, 2025
Merged

Feature/store transaction#131
zonotope merged 13 commits intomainfrom
feature/store-transaction

Conversation

@zonotope
Copy link
Contributor

This patch adds functions to transform consensus events to store any transactions they contain to connection storage and replace those transactions with transaction addresses. It also changes the standalone transactor to check if any transactions need to be resolved from storage (and resolve them if necessary) before attempting to process consensus events. It also simplifies the transaction/create handlers and the transaction processing to rely more on the default behavior of the db library instead of checking whether ledgers exist, and stage and committing transactions instead of transacting directly here when that's already done by the db library.

@zonotope zonotope requested a review from a team April 14, 2025 17:27
deref!)
commit-result (deref! (fluree/apply-stage! ledger staged-db))]
opts* (update opts :meta ensure-file-reports)
commit-result (deref! (fluree/transact! conn txn opts*))]
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice to see we're finally using this api in server!

Copy link
Contributor

@dpetran dpetran left a comment

Choose a reason for hiding this comment

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

🏪

@zonotope zonotope merged commit 9c8affc into main Apr 16, 2025
5 checks passed
@zonotope zonotope deleted the feature/store-transaction branch April 16, 2025 02:27
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