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

Equinox.Cosmos: Reinstate Tip isa Batch #110

Closed
wants to merge 1 commit into from
Closed

Conversation

bartelink
Copy link
Collaborator

The CosmosDb Stored Procedure in 1.x and 2.x guarantees to to never to mutate a document.

This was considered to be a critical guarantee on which internal projection systems would depend.

I'm of the opinion that the At-Least-Once-delivery characteristics of anything consuming from the CosmosDb Change Feed imply any projector can yield repeat sequences that are no different to those one might anticipate were mutation of events taking place.

As a result, the main thing that's necessary to consider exposing this facility is to ensure the following are adhered to:

  • Downstreams should not exclude documents with id="-1"
  • Downstream projectors should consider removing duplicate events where they are absolutely certain they have already projected an item in the current sessions (as noted above, consumers should already be able to handle partial resends of earlier events, including patterns such as 012314345678129, but this change will certainly make repeats more prevalent(so implementing this may be avoidable))

@bartelink
Copy link
Collaborator Author

Resolves #109

@bartelink bartelink force-pushed the master branch 2 times, most recently from 56e6be6 to 092be3d Compare March 16, 2019 22:29
@bartelink bartelink added this to the 3.0 milestone Apr 6, 2019
@bartelink bartelink force-pushed the master branch 2 times, most recently from 4fb1a89 to 7728239 Compare August 2, 2019 03:41
@bartelink bartelink force-pushed the master branch 2 times, most recently from d178a75 to e05e258 Compare August 15, 2019 23:47
@bartelink bartelink force-pushed the master branch 5 times, most recently from ac7e783 to 4305b88 Compare March 23, 2020 15:38
@bartelink bartelink mentioned this pull request Oct 1, 2020
4 tasks
@bartelink bartelink force-pushed the master branch 2 times, most recently from cd5febc to e92891c Compare October 9, 2020 08:07
@bartelink
Copy link
Collaborator Author

Superseded by #251

@bartelink bartelink closed this Oct 17, 2020
@bartelink bartelink deleted the tip-isa-batch branch June 2, 2021 10:10
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