-
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
Throw if no onError handler specified #198
Comments
To back this up, the relevant section from the Rx Design Guildelines says: "E.g. when calling the Subscribe method that only has an onNext argument, the OnError behavior will be to rethrow the exception on the thread that the message comes out from the observable sequence. The OnCompleted behavior in this case is to do nothing." (v1.0, section 5.2, p11) |
Is this closed with the above PR? |
I noticed this went into 0.9.1 👍 |
ReactiveX#198 As per Rx Design Guidelines 5.2: "when calling the Subscribe method that only has an onNext argument, the OnError behavior will be to rethrow the exception on the thread that the message comes out from the observable sequence. The OnCompleted behavior in this case is to do nothing." A new OnErrorNotImplementedException was created so it is explicit as to where the exception is coming from and why.
Add Maybe and Completable RateLimiter operator.
If an onError handler is not provided we should throw an exception if an error occurs even though it will likely show up on some other thread.
The text was updated successfully, but these errors were encountered: