Skip to content

Conversation

@ngocnhan-tran1996
Copy link
Contributor

No description provided.

Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
Copy link
Contributor

@cppwfs cppwfs left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution!
Just one small nitpick. Once corrected we can merge.

You can't use the Spring JMS namespace element `<jms:listener-container/>` to configure a container reference for the `<int-jms:message-driven-channel-adapter>` since that element doesn't actually reference a container.
Each `<jms:listener/>` sub-element gets its own `DefaultMessageListenerContainer` (with shared attributes defined on the parent `<jms:listener-container/>` element).
You can give each listener subelement an `id`, and use that to inject into the channel adapter, however, the `<jms:/>` namespace requires a real listener.
You can give each listener sub-element an `id`, and use that to inject into the channel adapter, however, the `<jms:/>` namespace requires a real listener.
Copy link
Contributor

Choose a reason for hiding this comment

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

For consistency subelement should not be hyphenated.

Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
@cppwfs
Copy link
Contributor

cppwfs commented Dec 2, 2025

Thank you for your contribution!

@cppwfs cppwfs merged commit 41e28dc into spring-projects:main Dec 2, 2025
3 checks passed
@ngocnhan-tran1996 ngocnhan-tran1996 deleted the polish-document branch December 2, 2025 16:10
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.

2 participants