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.

Sign up for GitHub

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

Jump to bottom

Adding observe to Clojure adapter. #171

Closed
wants to merge 1 commit into from
Closed

Adding observe to Clojure adapter. #171

wants to merge 1 commit into from

Conversation

josephwilk
Copy link
Contributor

Adding to the Clojure bindings observe which works much like queue but returns an Observable.

Like queue and execute it hides having to explicitly instantiate the HystrixExecutable.

@cloudbees-pull-request-builder
Copy link

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

@daveray
Copy link
Contributor

daveray commented Aug 12, 2013

Thanks. FWIW, there's already an pull request in the queue for this: #170

It has a few more features and more extensive tests. If you have any feedback, I'd love to hear it.

@josephwilk josephwilk closed this Aug 12, 2013
@josephwilk josephwilk deleted the clj-adapter-observe branch August 12, 2013 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@josephwilk @cloudbees-pull-request-builder @daveray