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

Modify FateStore to introduce a Seeder API #5374

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cshannon
Copy link
Contributor

@cshannon cshannon commented Mar 1, 2025

This adds a new Seeder API that can be used to seed transactions with a FateKey. The purpose of this new API is to allow for eventual batching of multiple seeding attempts together into one conditional mutation to improve performance. This change just updates the api and all calls to the seeder to attempt seeding will be performed individually. A future update will support the new functionality of batching and commiting all the changes together when the Seeder is closed.

This change is for Part 1 of #5160

This adds a new Seeder API that can be used to seed transactions with a
FateKey. The purpose of this new API is to allow for eventual batching
of multiple seeding attempts together into one conditional mutation to
improve performance. This change just updates the api and all calls to
the seeder to attempt seeding will be performed individually. A future
update will support the new functionality of batching and commiting all
the changes together when the Seeder is closed.

This change is for Part 1 of apache#5160
@cshannon cshannon requested a review from keith-turner March 1, 2025 20:18
@cshannon cshannon self-assigned this Mar 1, 2025
@cshannon cshannon added this to the 4.0.0 milestone Mar 1, 2025
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