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

Update comparator for Synchronized Sequence to be safer with distributed systems #895

Merged
merged 3 commits into from
Oct 23, 2019

Conversation

duro1
Copy link
Contributor

@duro1 duro1 commented Oct 9, 2019

Provides some level of security over duplicated commits when comparing shadows
Without this additional criteria, shadows with the same commit id will get the commit metadata from the earliest commit, rather than the commit associated to the shadow object

#894

ChadDurovec added 3 commits October 9, 2019 14:35
…ted systems

Provides some level of security over duplicated commits when comparing shadows
Without this additional criteria, shadows with the same commit id will get the commit metadata from the earliest commit, rather than the commit associated to the shadow object
@bartoszwalacik bartoszwalacik merged commit 0389fbf into javers:master Oct 23, 2019
@bartoszwalacik
Copy link
Member

thanks @duro1

@bartoszwalacik
Copy link
Member

btw, your commits are not linked to your github account, if you link them, you will be automatically counted as javers contributor

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