Skip to content

Add test cases for unsubscribe of Hystrix commands before terminal events#1198

Merged
mattrjacobs merged 2 commits intoNetflix:masterfrom
mattrjacobs:unsubscribe-from-hystrix-observable-command
May 5, 2016
Merged

Add test cases for unsubscribe of Hystrix commands before terminal events#1198
mattrjacobs merged 2 commits intoNetflix:masterfrom
mattrjacobs:unsubscribe-from-hystrix-observable-command

Conversation

@mattrjacobs
Copy link
Copy Markdown
Contributor

3 fixes in this PR:

  • execution semaphore was not being released
  • fallback semaphore was not being released
  • isExecutionComplete was not set to true

@mattrjacobs mattrjacobs merged commit dafbded into Netflix:master May 5, 2016
@mattrjacobs mattrjacobs deleted the unsubscribe-from-hystrix-observable-command branch May 5, 2016 04:13
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.

1 participant