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

Fix cluster-level error marshalling #4278

Merged
merged 1 commit into from
Sep 30, 2015
Merged

Fix cluster-level error marshalling #4278

merged 1 commit into from
Sep 30, 2015

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Sep 30, 2015

This change corrects use of the error-message lookup table, so that errors that are marshalled just as strings across the cluster are correctly recognised on the receiving nodes. This allows the higher-level code to continue to use error types such as ErrRetentionPolicyNotFound

This fix required that certain error prefixes be removed from error handling, and that errors are passed through cleanly.

Fixes #4267

@otoolep otoolep changed the title Fix error marshalling Fix cluster-level error marshalling Sep 30, 2015
@otoolep
Copy link
Contributor Author

otoolep commented Sep 30, 2015

@benbjohnson

@otoolep otoolep self-assigned this Sep 30, 2015
@otoolep otoolep added this to the 0.9.5 milestone Sep 30, 2015
@benbjohnson
Copy link
Contributor

👍

otoolep added a commit that referenced this pull request Sep 30, 2015
Fix cluster-level error marshalling
@otoolep otoolep merged commit 83f0d9b into master Sep 30, 2015
@otoolep otoolep deleted the error_marshalling branch September 30, 2015 20:18
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.

Cluster-level errors not marshalled as expected
2 participants