+1 for removing the old rxjava support I would also be fine with it living in a separate module.
Christian 2017-11-15 16:56 GMT+01:00 Sergey Beryozkin <sberyoz...@gmail.com>: > Hi > > cxf-rt-rs-extension-rx ships the code for both (old) RxJava and RxJava2 > code. It supports returning RxJava2 Flowable and Observable on the server > and accepting it on the client, and the same for the (old) RxJava > Observable... > > While even the (old) RxJava code is very new for CXF, the reality is that > RxJava has been around for a while now and with RxJava2 embracing > org.reactivestreams, it's hard to see CXF users preferring to start with > the (old) RxJava. > > The other minor problem is that cxf-rt-rs-extension-rx has optional RxJava > and RxJava2 deps to be able to ship the relevant code in the same module > and splitting it into 2 modules will be too much at this point. > > I suggest that unless some users confirm (I CC to the users) that they > need to use the (old) RxJava code, then we just remove it and make things > much simpler... > > Thanks, Sergey > -- -- Christian Schneider http://www.liquid-reality.de <https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e46&URL=http%3a%2f%2fwww.liquid-reality.de> Computer Scientist http://www.adobe.com
