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

Fix ObserveOn and add ParallelMerge Scheduler overload #502

Merged

Conversation

benjchristensen
Copy link
Member

  • ObserveOn was not correctly recursing when the source was async. It would create a new scheduler (i.e. a new thread) each time.
  • Also added unit tests to ParallelMerge

- ObserveOn was not correctly recursing when the source was async. It would create a new scheduler (i.e. a new thread) each time.
- Also added unit tests to ParallelMerge
@benjchristensen
Copy link
Member Author

This is debugged and pair programmed with @headinthebox

benjchristensen added a commit that referenced this pull request Nov 20, 2013
Fix ObserveOn and add ParallelMerge Scheduler overload
@benjchristensen benjchristensen merged commit 30b6b08 into ReactiveX:master Nov 20, 2013
@benjchristensen benjchristensen deleted the observeOn-parallelMerge branch November 20, 2013 00:31
@cloudbees-pull-request-builder

RxJava-pull-requests #426 SUCCESS
This pull request looks good

rickbw pushed a commit to rickbw/RxJava that referenced this pull request Jan 9, 2014
…lelMerge

Fix ObserveOn and add ParallelMerge Scheduler overload
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.

None yet

2 participants