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

Introduce record version history for optimistic store #618

Conversation

sav007
Copy link
Contributor

@sav007 sav007 commented Aug 9, 2017

Rolling back optimistic updates is smart now, it keeps the history of record updates.
If mutation finished it's execution it rolls back only its version of the record and keeps the rest versions for still running mutations.

Closes #583

@sav007 sav007 force-pushed the feature-583/optimistic-store-record-journal branch 3 times, most recently from 5e600d6 to d34addb Compare August 10, 2017 02:38
Rolling back optimistic updates is smart now, it keeps the history of record updates.
If mutation finished it's execution it rolls back only its version of the record and keeps the rest versions for still running mutations.

Closes apollographql#583
@sav007 sav007 force-pushed the feature-583/optimistic-store-record-journal branch from d34addb to f22f3f1 Compare August 10, 2017 03:36
return changedCacheKeys;
}

private final class RecordJournal {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like the name 👍

@sav007 sav007 merged commit b1eaaf3 into apollographql:master Aug 10, 2017
@sav007 sav007 deleted the feature-583/optimistic-store-record-journal branch August 10, 2017 16:26
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.

3 participants