-
Notifications
You must be signed in to change notification settings - Fork 392
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
Bring 2.0 bug fixes to 2.2 #901
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… the same lastmod
If we call riak_core_vnode_master:sync_spawn_command, it ends up sending a gen_event all-state event to the proxy for forwarding. However, in handle_overload, the old code only had case clauses for '$gen_event' and not '$gen_all_state_event'. This means the event would be passed to handle_overload_info instead of handle_overload_request, which would likely skip the vnode callback code that sends a response. In Riak this meant that certain operations would hang during overload, since the caller to sync_spawn_command would never be sent a response. (This situation was made worse by the fact that the call is being done with an infinity timeout, but that's an issue we'll have to address some other time.)
There have been a few issues where rapidly updating the ring results in sub optimal behavior. These have been difficult to debug due to the lack of logging in the `riak_core_ring` module. We’ve added debug logging which can be enabled as needed to log ring metadata merges(buckets, real, etc) and ring membership and resulting rings from a reconciliation. For 2.2.1, we should add a new lager sink just for logging ring operations.
…orrectly named dropped_vnode_requests_total, which is the stats projection
Fix RIAK-2169 by fixing stat name `dropped_vnode_requests`
Fix RIAK-2643 by adding debug logging for meta and ring
Fix potential hang due to vnode proxy overload bug [JIRA: RIAK-3124]
add nodenames to the merge_meta comparison
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.