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(Cosmos): Handle blind write of events+unfolds idempotently #463

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bartelink
Copy link
Collaborator

Stored proc extension to remove redundant reads when writers such as CosmosStoreSink are syncing unfolds and events

Leaving as a draft for now for the following reasons:

  • not a massive perf win
  • existing API returns the conflicting events; ideally the writer should verify that the events are equivalent (if someone was using the EventsContext API and not working based on cached etags, then you'd want something equivalent to the EventStoreDB EventId checks)
  • the more complete answer is to tag the unfolds with a revision value, and have the idempotency be based on that matching and/or unfolds being completely empty (as it stands we don't have a good way to nullify the write costs when retraversing)

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