Update to RxJava 2 - Githubissues

alaisi / postgres-async-driver

Asynchronous PostgreSQL Java driver

Apache License 2.0

287 stars 38 forks source link

Closed bnorm closed 5 years ago

bnorm commented 7 years ago

Fixes: #47

Some notes on the changes:

I made use of Single and Completable where I felt appropriate. Since this is a pretty big API change, I thought it would be best to convert stuff to something more representative of the operation. (Plus, RxJava 2 does not allow nulls)
I changed a few custom lift operations to common operators. I'm pretty confident I did this correctly as all the unit tests passed.

More than happy to update anything, just let me know.

iferca commented 6 years ago

@alaisi Any plans to review this PR? How about that core library and wrap it using Rx or Reactor, have something started that can be continued?