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

verify_listkeys failure #149

Merged
merged 1 commit into from
Jan 30, 2013
Merged

Conversation

engelsanchez
Copy link
Contributor

Getting an "insufficient vnodes" message. I used to see that a lot, probably now infrequent. The logs:
http://basho-giddyup.herokuapp.com/#/projects/riak/scorecards/7/7-271/2079

@engelsanchez
Copy link
Contributor Author

@ghost ghost assigned engelsanchez Jan 25, 2013
@engelsanchez
Copy link
Contributor Author

Latest insights: it seems to happen when querying the node that has just joined the cluster. It seems ownership of the data has not been negotiated yet and that node can not provide a coverage plan for the listkeys command.

This seems to be the same as basho/riak_core#228

Fixes issue #149, related to basho/riak_core#228
If queried right after it has joined the cluster, a node may not have
the other nodes registered by the node watcher yet. Switching to waiting
until all nodes in the cluster have been registered by the node watcher
before performing query.
Also, querying for all nodes after stopping/starting just in case,
although I haven't seen problems there, but sound reasonable.
Removing assert from the call to wait_until_no_pending_changes since it
is already asserting internally.
@seancribbs
Copy link
Contributor

👍 Reads sane AND passes.

engelsanchez added a commit that referenced this pull request Jan 30, 2013
@engelsanchez engelsanchez merged commit 8dcb0d1 into master Jan 30, 2013
@engelsanchez engelsanchez deleted the eas-fix-insufficient-nodes-listkeys branch January 30, 2013 16:15
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