add feign-rxjava

[codefromthecrypt]

This should support at least an Observer implementation (plus other ideas @benjchristensen has in mind)

[benjchristensen]

I'm definitely interested in working on this module with you.

[codefromthecrypt]

this might be best resurrected as feign-rxnetty, possibly using the bolt-on idea from @jdamick.

Only thing is that the interactions could be complex. ex. the builder for feign-rxnetty probably wouldn't want to expose many of the existing things that builders accept. Ex. Client (as it is synchronous).

Probably first stage would be to have a buddy module with a hard dep on rxnetty, and exposes an alternate builder that only exposes things that actually make sense. builders aren't that much code and heck, google auto can make them for us.

cc @brianm @NiteshKant

[spencergibb]

+1

[jdamick]

+1

[yterradas]

+1

[stemcd]

+1

[christopherlakey]

👍

[codefromthecrypt]

ps. those interested, have a look at the new hystrix module, which implicitly support rxjava. It is likely rxjava support will integrate similarly https://github.com/Netflix/feign/tree/master/hystrix

[mariuszs]

👍

[christopherlakey]

Methods that do not return HystrixCommand are still wrapped in a HystrixCommand, but execute() is automatically called for you.

It might be nice if methods that return Observable are still wrapped in a HystrixCommand, but toObservable() is automatically called for you. It seems this would be a straight forward feature to add.

Once an RxJava module is added, you could choose either the RxJava module or the Hystrix module w/o having to update your interfaces.

I don't have any Hystrix experience yet, but I would be willing to implement the required changes if there's a chance of a PR being accepted.

[christopherlakey]

Just submitted a PR with Observable support and updated unit tests.

#307