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

Upgrade RxJava2 to 2.1.14, Upgrade AssertJ to 2.9.1 #2771

Closed
wants to merge 1 commit into from

Conversation

bednar
Copy link

@bednar bednar commented May 31, 2018

This Pull request also cover the RxJava2 API changes between 2.0.0 and 2.1.14

  • 2.0.6: introduces specific exception wrappers to help distinguish and track down what was happening the time of the error = exceptions are wrapped in UndeliverableException - additional info
  • 2.0.7: a non-positive Subscription.request() will not stop the current stream but signal an error via RxJavaPlugins.onError - additional info

Hi @JakeWharton, it is correct change for you?

@jabson009
Copy link

Good one

@JakeWharton
Copy link
Member

Thanks, I landed a derivative of this change in #3636 which also removes the runtime backpressure support and completely removes the associated test cases.

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.

3 participants