Skip to content
This repository has been archived by the owner on Mar 7, 2020. It is now read-only.

Add RxJava2 Support #12

Open
kdavisk6 opened this issue Aug 16, 2018 · 4 comments
Open

Add RxJava2 Support #12

kdavisk6 opened this issue Aug 16, 2018 · 4 comments
Labels

Comments

@kdavisk6
Copy link
Member

Similar to Reactive, we should support RxJava2 and their Streams implementation.

@kptfh
Copy link
Collaborator

kptfh commented Aug 17, 2018

What about to wrap Reactor core implementation with reactor.adapter.rxjava.RxJava2Adapter to provide RxJava2 types support? Will it work for Android clients?

@kdavisk6
Copy link
Member Author

That is one approach to this, but wouldn't that require both reactor and rxjava2 to work?

With regards to Android, rxjava2 is recommended, but reactor should work if the SDK version is higher than 26. See Reactor Prerequisites

If we want to support Android, I think we should keep rxjava2 and reactor explicitly separate.

@kptfh
Copy link
Collaborator

kptfh commented Aug 17, 2018

From my side I can offer to follow (and may try to implement) this approach (wrapper over Reactor) as not sure how fast pure RxJava2 version will be implemented (guess it may take years :( ).

@kdavisk6
Copy link
Member Author

We can help with that. These issues are not tasks for you to take on, only to keep track of what I think would be good features for use to consider.

@velo velo added the API label Aug 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants