@Matt no problem at all.. just making it clear...
And I am actually conflicted if there would be value in bringing it to artemis. If there would be any extra features to be developed? I don't honestly "believe" in rest over messaging.... the API gets too stateful to my taste. that's why I'm looking for guidance. On Tue, Sep 13, 2022 at 11:23 AM Matt Pavlovich <mattr...@gmail.com> wrote: > > Hey Clebert- > > Yeah, I was just addressing Lucas’ question from earlier in the thread re > removing from ActiveMQ 5. > > -Matt > > > On Sep 13, 2022, at 9:27 AM, Clebert Suconic <clebert.suco...@gmail.com> > > wrote: > > > > I'm not suggesting we remove it.. just if there is value in bringing > > it to artemis. > > > > On Tue, Sep 13, 2022 at 10:24 AM Matt Pavlovich <mattr...@gmail.com> wrote: > >> > >> The REST service in ActiveMQ 5 is actively used, and should not be > >> removed. If there are bugs or issues, please file a JIRA. > >> > >> Thanks, > >> Matt Pavlovich > >> > >>> On Sep 12, 2022, at 3:58 PM, Tetreault, Lucas > >>> <tetlu...@amazon.com.INVALID> wrote: > >>> > >>> Should we remove REST from ActiveMQ "Classic" as well? I think the same > >>> arguments apply about it being abandonware, etc. > >>> > >>> We could deprecate it in the upcoming 5.18.0 release and use this as > >>> incentive to cut a 6.0.0 release? __ That would be exciting! > >>> > >>> - Lucas > >>> > >>> On 2022-09-12, 7:41 AM, "Clebert Suconic" <clebert.suco...@gmail.com> > >>> wrote: > >>> > >>> CAUTION: This email originated from outside of the organization. Do not > >>> click links or open attachments unless you can confirm the sender and > >>> know the content is safe. > >>> > >>> > >>> > >>> I will go ahead and remove it... > >>> > >>> > >>> I will also bump upstream/main as 3.0 as part of the removal. > >>> > >>> On Thu, Sep 8, 2022 at 2:24 PM Clebert Suconic > >>> <clebert.suco...@gmail.com> wrote: > >>>> > >>>> I didn’t meant to imply I was going to remove it now > >>>> > >>>> I intend to do it on Monday. If no objection. > >>>> > >>>> > >>>> Although keeping it means we would have to fix it. I honestly don’t > >>>> see many options to keep it. > >>>> > >>>> On Thu, Sep 8, 2022 at 1:09 PM Robbie Gemmell <robbie.gemm...@gmail.com> > >>>> wrote: > >>>>> > >>>>> While I'm fine with it going I'd maybe give other folks more of chance > >>>>> to reply...or at least use lay concensus style 'ill do it at <date> if > >>>>> noone objects' :) > >>>>> > >>>>> On Thu, 8 Sept 2022 at 17:48, Clebert Suconic > >>>>> <clebert.suco...@gmail.com> wrote: > >>>>>> > >>>>>> I will go ahead and remove it... > >>>>>> > >>>>>> > >>>>>> to be honest I don't believe much in rest from JMS due to the session > >>>>>> and stateful nature. > >>>>>> > >>>>>> > >>>>>> But if we were to provide REST for our users, I would rather bring the > >>>>>> servlet from AMQ5. it would be a major task anyway... and this > >>>>>> module has to go for sure. > >>>>>> > >>>>>> On Thu, Sep 8, 2022 at 10:20 AM Robbie Gemmell > >>>>>> <robbie.gemm...@gmail.com> wrote: > >>>>>>> > >>>>>>> It has looked to be rotting for a long time, and requires various user > >>>>>>> hoop jumping I dont expect many/any folks are interested in doing....I > >>>>>>> think removing it makes sense. > >>>>>>> > >>>>>>> On Thu, 8 Sept 2022 at 14:54, Clebert Suconic > >>>>>>> <clebert.suco...@gmail.com> wrote: > >>>>>>>> > >>>>>>>> I'm not sure if there's much to discuss here. Rest in Artemis has > >>>>>>>> been > >>>>>>>> abandonware for a while (like 5 years)... The jboss-rest interface is > >>>>>>>> a few major releases behind, the module compiles but it's not > >>>>>>>> functional, and any time someone ask questions we just mention don't > >>>>>>>> use it... (favoring stomp instead). > >>>>>>>> > >>>>>>>> > >>>>>>>> https://github.com/apache/activemq-artemis/tree/main/artemis-rest > >>>>>>>> > >>>>>>>> > >>>>>>>> As part of new logging changes, we are moving activemq-artemis into > >>>>>>>> 3.0... > >>>>>>>> > >>>>>>>> At this point I see no other choice than remove the module. > >>>>>>>> > >>>>>>>> Any objections? > >>>>>>>> > >>>>>>>> -- > >>>>>>>> Clebert Suconic > >>>>>> > >>>>>> > >>>>>> > >>>>>> -- > >>>>>> Clebert Suconic > >>>> > >>>> -- > >>>> Clebert Suconic > >>> > >>> > >>> > >>> -- > >>> Clebert Suconic > >>> > >> > > > > > > -- > > Clebert Suconic > -- Clebert Suconic
