Skip to content

Conversation

marcinwieckowski
Copy link

Adds a new option on the JMS Component configuration: 'replyCorrelationProperty'.

Difference between this and 'correlationProperty'" is that 'correlationProperty' tells which request property holds the correlation id value, and it does not affect the selector for the reply (JMSCorrelationID=), while 'replyCorrelationProperty' tells which reply property will hold the correlation id value, and it does affect the selector for the reply (=).

Use case: reply provider has custom correlation behaviour where it uses custom headers (not JMSMessageID and JMSCorrelationID) for correlation.

Copy link
Contributor

github-actions bot commented Oct 6, 2025

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run

  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot.

  • You can label PRs using build-all, build-dependents, skip-tests and test-dependents to fine-tune the checks executed by this PR.

  • Build and test logs are available in the Summary page. Only Apache Camel committers have access to the summary.

  • ⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@marcinwieckowski
Copy link
Author

Sorry, I had some problems with generating all the necessary auto-generated files. Now it should be ok.

@davsclaus
Copy link
Contributor

CAMEL-22500

@davsclaus davsclaus force-pushed the jms-new-reply-correlation-property branch from 12f7f73 to 903459b Compare October 7, 2025 06:57
davsclaus added a commit that referenced this pull request Oct 7, 2025
…D selector for InOut.

Thanks to Marcin Więckowski for the pr: #19446

New JMS component option 'replyCorrelationProperty'.
Commit changes outside components/camel-jms
Fix description for replyCorrelationProperty (<> -> &lt;&gt;)
Commit changes outside components/camel-jms
Commit changes outside components/camel-jms
Commit changes outside components/camel-jms
@davsclaus
Copy link
Contributor

Thanks for the PR. I have manually merged this. Yeah its a bit difficult due to those source files are regenerated in various places. So closing this PR as the code has been manually merged.

@davsclaus davsclaus closed this Oct 7, 2025
@marcinwieckowski
Copy link
Author

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants