[jira] [Work logged] (AMQ-9219) Add activemq-client-jakarta transition module for 5.17.x

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/AMQ-9219?focusedWorklogId=845478&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-845478 ]

ASF GitHub Bot logged work on AMQ-9219:
---------------------------------------

                Author: ASF GitHub Bot
            Created on: 14/Feb/23 20:31
            Start Date: 14/Feb/23 20:31
    Worklog Time Spent: 10m 
      Work Description: cshannon commented on PR #969:
URL: https://github.com/apache/activemq/pull/969#issuecomment-1430337307

   So looking at this and thinking about it more I am seeing several problems including the fact that I realized this doesn't actually solve the problem for Spring boot compatibility.
   
   1. This doesn't really make sense to introduce a new module like this into 5.17.x for a point/bug fix release as 5.17.x has already been out a while. Furthermore this only copies JMS 1.1 classes so is completely broken for most of the methods. This should be targeted for a future version.
   
   2. In main/5.18.x there has been more JMS 2.0 work done so we should be using that as the baseline for this because it handles things properly like working between vendors. But there's still more work to be done because not everything is implemented. Any of the API methods that can be implemented now probably should be. I think most things besides shared subs can probably be done.
   
   3. Anyone that uses the Jakarta API could reasonably assume that calling any method should work. Since there is no previous Jakarta messaging version before 3.0 any code that is Jakarta compatible could assume the spec should work so we would need to finish the implementation or at least be very clear in the documentation what won't work.
   
   4. This doesn't actually fix the issue for Spring Boot 3 Spring Boot won't be able to add back in support because Spring boot is going to pull in the server itself to allow running the 5.x broker and the broker doesn't use the new Jakarta API. A new Jakarta client would only work if using just the client in the JVM and not the broker itself. For Spring 6.x I'm not sure, it may be good enough if it's just client support but if Spring 6 is calling off to any new methods that are not implemented in their messaging module (which they reasonably can do as I pointed out in number 3) then this breaks and won't work.
   




Issue Time Tracking
-------------------

            Worklog Id:     (was: 845478)
    Remaining Estimate: 0h
            Time Spent: 10m

> Add activemq-client-jakarta transition module for 5.17.x
> --------------------------------------------------------
>
>                 Key: AMQ-9219
>                 URL: https://issues.apache.org/jira/browse/AMQ-9219
>             Project: ActiveMQ
>          Issue Type: Task
>            Reporter: Matt Pavlovich
>            Assignee: Matt Pavlovich
>            Priority: Major
>             Fix For: 5.17.4
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> This module throws UnsupportedOperationException for all JMS 2.0 methods



--
This message was sent by Atlassian Jira
(v8.20.10#820010)