Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: partial batches #243

Merged
merged 2 commits into from
Feb 27, 2023
Merged

feat: partial batches #243

merged 2 commits into from
Feb 27, 2023

Conversation

QuantumExplorer
Copy link
Collaborator

Issue being fixed or feature implemented

In RS-Drive atomicity of state transitions is paramount. A state transition is a collection of multiple operations that often will cause one or more identity balances to change. The amount that the identity balance must change by is driven by costs of changes in the batch. Ideally we should be able to alter the batch by adding operations based on calculated information coming from the batch itself.

For example in one batch have the document insertion, but also the reduction of credits of the identity due to the document insertion.

What was done?

To be able to perform this feature, we need to get back costs from an insertion, then perform just in time updates to the batch, this is what is coded up in this PR.

How Has This Been Tested?

Added some unit tests

Breaking Changes

No breaking changes

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@QuantumExplorer QuantumExplorer merged commit 8c547b3 into develop Feb 27, 2023
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.

None yet

2 participants