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 for issue #332 (Hystrix Javanica Error Propagation does not work cor... #334

Merged
merged 1 commit into from
Nov 11, 2014

Conversation

dmgcodevil
Copy link
Contributor

...rectly)

@cloudbees-pull-request-builder

Hystrix-pull-requests #163 FAILURE
Looks like there's a problem with this pull request

@dmgcodevil
Copy link
Contributor Author

The build inherited the failed test case since last pull request. The test case is inside hystrix-core, which wasn't modified in this pull request (only at hystrix-javanica).

@benjchristensen
Copy link
Contributor

Are these breaking changes for hystrix-javanica API? If so do we need to handle the release differently?

benjchristensen added a commit that referenced this pull request Nov 11, 2014
Fix for issue #332 (Hystrix Javanica Error Propagation does not work cor...
@benjchristensen benjchristensen merged commit 3f8e2b5 into Netflix:master Nov 11, 2014
@dmgcodevil
Copy link
Contributor Author

No, this changes don't affect API. After the fix the feature must work properly.

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