-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Single.flatMapPublisher full implementation #6015
Comments
Sure. |
In case you haven't started yet, you may want to wait for #6017 so the changes can be benchmarked. |
That's brilliant, thanks |
A question about fusion. I notice that Should I chase fusion for this operator? Any suggestions? |
Fusion won't work here due to the delayed subscription to the Flowable. Don't bother with it. |
Righto, ta. |
I noticed
Single.flatMapPublisher
is implemented astoFlowable().flatMapPublisher(...)
which subjects the one flowable to batched requests. I'd like to have requests go straight through to the one flowable. Can I contribute this operator?The operator at the moment is at https://github.com/davidmoten/rxjava2-http/blob/master/src/main/java/org/davidmoten/rx2/io/internal/FlowableSingleFlatMapPublisher.java without fusion. I can add more tests, fusion (transitive as per
doOnEach
) and follow the projects's naming and visibility conventions.The text was updated successfully, but these errors were encountered: