Skip to content

Comments

[AMQ-8325] JMS 2.0 XA supported operations#1044

Merged
mattrpav merged 1 commit intoapache:mainfrom
mattrpav:AMQ-8325
Sep 19, 2023
Merged

[AMQ-8325] JMS 2.0 XA supported operations#1044
mattrpav merged 1 commit intoapache:mainfrom
mattrpav:AMQ-8325

Conversation

@mattrpav
Copy link
Contributor

No description provided.

@mattrpav mattrpav self-assigned this Aug 30, 2023
@mattrpav mattrpav marked this pull request as ready for review September 15, 2023 16:35
@mattrpav mattrpav requested a review from cshannon September 15, 2023 16:35
@cshannon
Copy link
Contributor

Is the plan to try and get this into 6.0.0 or for 6.1.0?

@mattrpav
Copy link
Contributor Author

@cshannon Yes, I was looking to get it into 6.0.0/5.18.NEXT

The change is minor (all wrapper, no functional).

Thoughts?

@cshannon
Copy link
Contributor

@cshannon Yes, I was looking to get it into 6.0.0/5.18.NEXT

The change is minor (all wrapper, no functional).

Thoughts?

Yeah that's fine I wasn't sure how soon we were planning to release and just didn't want to put anything in last second but there should be time

Copy link
Contributor

@cshannon cshannon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took a quick glance and this looks ok to me since it's just a wrapper and delegating to existing functionality.

@jbonofre jbonofre self-requested a review September 19, 2023 13:29
@jbonofre
Copy link
Member

I think this wrapper is great for 6.0.0 and next 5.18.x. Thanks !

@mattrpav mattrpav merged commit 09ae41a into apache:main Sep 19, 2023
@mattrpav mattrpav deleted the AMQ-8325 branch September 19, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants