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

Added support for Request Cache and Reactive Execution #229

Merged
merged 1 commit into from
Apr 1, 2014
Merged

Added support for Request Cache and Reactive Execution #229

merged 1 commit into from
Apr 1, 2014

Conversation

dmgcodevil
Copy link
Contributor

Major changes:
AsyncCommand -> AsyncResult
Added support for Request Cache and Reactive Execution
Ability use @HystrixCollapser and @HystrixCommand together
Code improvements, updated doc.
LazyCommandExecutionAction (improve performance)

@cloudbees-pull-request-builder

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

@dmgcodevil
Copy link
Contributor Author

Something wrong with this test, strange but it works on my local machine.

@dmgcodevil
Copy link
Contributor Author

Could you merge it in master (1.4.x) too or please let me know if you need new PR for master

AsyncCommand -> AsyncResult
Added support for Request Cache and Reactive Execution
Ability use @HystrixCollapser and @HystrixCommand together
Code improvements, updated doc.
LazyCommandExecutionAction (improve performance)
@dmgcodevil
Copy link
Contributor Author

I've created PR in master. I changed tests code to not use deprecated methods. You can merge this PR to if you don't want merge this changes from 1.3.x to master.

@dmgcodevil
Copy link
Contributor Author

Hi guys, any updates here ?

@benjchristensen
Copy link
Contributor

Just been busy ... I'll get to it.

benjchristensen added a commit that referenced this pull request Apr 1, 2014
Added support for Request Cache and Reactive Execution
@benjchristensen benjchristensen merged commit 7df30d3 into Netflix:1.3.x Apr 1, 2014
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