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(get): get from quorum #866

Merged
merged 1 commit into from Jul 9, 2014

Conversation

4 participants
@xiang90
Copy link
Contributor

commented Jun 23, 2014

@bmizerany @philips

I do not suggest people to use this function unless they know exactly what they want.
This is just for linearizability in real time order. Index is a better indication of the actual order of events.
Nothing to do with etcd's write consensus/consistency model.

@bmizerany

This comment has been minimized.

Copy link
Contributor

commented Jun 23, 2014

LGTM. It agree it needs a warning in the docs.

@yichengq

This comment has been minimized.

Copy link
Contributor

commented Jun 23, 2014

lgtm. A smoke test for it would be great.

@philips

This comment has been minimized.

Copy link
Contributor

commented Jun 23, 2014

We need to add documentation for this and figure out what happens in a mixed version cluster.

philips added a commit that referenced this pull request Jul 9, 2014

Merge pull request #866 from coreos/qread
feat(get): get from quorum

@philips philips merged commit 1cffdb3 into master Jul 9, 2014

@yichengq yichengq deleted the qread branch Aug 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.