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

Adding utility functions for observables of strings useful for processing non blocking IO. #457

Closed
wants to merge 31 commits into from

Conversation

abersnaze
Copy link
Contributor

I started developing this for use with the rxjava-apache-http contrib module to safely and scalable decode infinite observables of byte arrays into strings but thought it should live in core because it could be useful for non blocking disk IO.

I did not make this part of rx.Observable because these operators only work on observables of string. The drawback is that discoverability will be harder.

@cloudbees-pull-request-builder

RxJava-pull-requests #376 FAILURE
Looks like there's a problem with this pull request

@benjchristensen
Copy link
Member

Open question ... should static utility classes like this be kept in core or kept external in contrib-modules?

These seems generic enough on Java primitives to be in core as opposed to Swing/Android which are kept in contrib modules.

I'd like other feedback before making a decision.

@karansinghkjs346
Copy link

hello everyone,
i am new to github and i want to start contributing , i am a student of Computer science engg final year with knowledge in java , data structure and networking , please guide me a bit i found this project and it seems like it matches according to my skills can anyone please help me to start
i will be grateful to you

@zsxwing
Copy link
Member

zsxwing commented Nov 1, 2013

@karansinghkjs346 you should ask such question in http://groups.google.com/d/forum/rxjava. Your question is irrelevant to this PR. However, wiki is very helpful for you. https://github.com/Netflix/RxJava/wiki/How-to-Contribute and https://github.com/Netflix/RxJava/wiki

Conflicts:
	rxjava-core/src/main/java/rx/observables/BlockingObservable.java
@cloudbees-pull-request-builder

RxJava-pull-requests #390 FAILURE
Looks like there's a problem with this pull request

@cloudbees-pull-request-builder

RxJava-pull-requests #391 FAILURE
Looks like there's a problem with this pull request

@zsxwing
Copy link
Member

zsxwing commented Nov 6, 2013

I have merged your second commit to PR #461. Thanks.

zsxwing and others added 20 commits November 19, 2013 13:09
Implement the scheduler overloads for Range, From, StartWith
from(scheduler) -> add diagram, standardize javadoc comments
range(scheduler) -> add diagram, standardize javadoc comments
startWith(scheduler) -> add diagram, standardize javadoc comments
- 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
…lelMerge

Fix ObserveOn and add ParallelMerge Scheduler overload
- make Eclipse and Java/Scala interop happy
Javadoc changes:
* add links to RxJava wiki
* correct obsolete <img> urls
* remove unneeded version string from Linq URLs
* standardize javadoc formatting
* a few other minor javadoc corrections
* @param expected
* @param actual
*/
public static <T> void assertObservableEquals(Observable<T> expected, Observable<T> actual) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do these perhaps belong in a TestObservable class rather than BlockingObservable?

@benjchristensen
Copy link
Member

This looks like useful stuff ... can you rebase since it can no longer merge cleanly? Also I wonder if the assert things should be in something other than BlockingObservable.

@cloudbees-pull-request-builder

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

@benjchristensen
Copy link
Member

That merge picked up a huge number of changes (66 files), can you please close this and rebase your changes onto a clean branch?

@abersnaze abersnaze closed this Nov 22, 2013
jihoonson pushed a commit to jihoonson/RxJava that referenced this pull request Mar 6, 2020
…aram plus some code refactoring (ReactiveX#457)

* change fallback package

* Review comments

* Review comments
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.

8 participants