digital-asset / daml

The Daml smart contract language
https://www.digitalasset.com/developers
791 stars 198 forks source link

Upgrade to rxjava 3.x #15180

Open ray-roestenburg-da opened 1 year ago

ray-roestenburg-da commented 1 year ago

Upgrade to latest rxjava 3.x.

The 2.x version is end-of-life as of February 28, 2021. No further development, support, maintenance, PRs and updates will happen. The Javadoc of the very last version, 2.2.21, will remain accessible.

See https://github.com/ReactiveX/RxJava

Depends on

stefanobaghino-da commented 1 year ago

A viable approach, as suggested initially by you off-band with a few points from me:

S11001001 commented 1 year ago

We should exclude any parts of bindings-rxjava that are currently deprecated from bindings-rxjava3 at the time of this copy.

S11001001 commented 1 year ago

I've come around to @stefanobaghino-da 's idea that we ought to clean up the overloads with builders or other argument-set representations before doing the rxjava3 split, and then delete the old deprecated functions in rxjava3 so we have a clean base.

stefanobaghino-da commented 1 year ago

I've come around to @stefanobaghino-da 's idea that we ought to clean up the overloads with builders or other argument-set representations before doing the rxjava3 split, and then delete the old deprecated functions in rxjava3 so we have a clean base.

Any specific reason why you'd favor my take?

S11001001 commented 1 year ago

@stefanobaghino-da So that we have a good stepped approach to simply not having the deprecated functions in rxjava3.

stefanobaghino-da commented 1 year ago

@stefanobaghino-da So that we have a good stepped approach to simply not having the deprecated functions in rxjava3.

Makes sense. I think it's cleaner to start with a library free of deprecations, as long as we can have a migration path.

stefanobaghino-da commented 1 year ago

I would also recommend evaluating whether something can be done for the overloads with a security token. I think a gRPC interceptor might be part of the solution.

ray-roestenburg-da commented 1 year ago

Sounds good