Skip to content

Conversation

@r-dunning
Copy link
Contributor

@r-dunning r-dunning commented Nov 12, 2024

Firing the Data Changed event where the event writes to the same stream in a transactional batch would result in conflicts as the next event to be processed in the batch would conflict on Ids. I've added support for the ContainerMock to take a param on update methods whether to immediately invoke a registered DataChanged event, or queue the invoke. The TestTransactionalBatch will update with the param set to Batch, then process the queued DataChanged events its self.
This also more closely aligns what would happen in cosmos as any CFPS are most likely to receive their events after the stream has already returned
Other fix:

  • Fix not firing data changed event on replace item

@r-dunning r-dunning closed this Nov 12, 2024
@r-dunning r-dunning reopened this Nov 12, 2024
@r-dunning r-dunning marked this pull request as draft November 12, 2024 16:03
@r-dunning r-dunning marked this pull request as ready for review November 13, 2024 12:08
@AButler AButler merged commit c553c50 into LogOtter:main Nov 15, 2024
1 check failed
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