Skip to content

CAMEL-19350: outputTarget in camel-jpa (3.x)#10079

Merged
davsclaus merged 7 commits intoapache:camel-3.xfrom
jcszymansk:CAMEL-19350-3.x
May 15, 2023
Merged

CAMEL-19350: outputTarget in camel-jpa (3.x)#10079
davsclaus merged 7 commits intoapache:camel-3.xfrom
jcszymansk:CAMEL-19350-3.x

Conversation

@jcszymansk
Copy link
Copy Markdown
Contributor

Description

Unlike camel-sql, camel-jpa does not allow the user to put the result elsewhere than the message body; if this is needed the user must: preserve the body, use the component, put the result in the desired destination, restore the body.

This PR adds the possibility to put the result into a header or property without destroying the message body.

Target

  • I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I formatted the code using mvn -Pformat,fastinstall install && mvn -Psourcecheck

@github-actions
Copy link
Copy Markdown
Contributor

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

⚠️ Please note that the changes on this PR may be tested automatically.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

@davsclaus
Copy link
Copy Markdown
Contributor

Can you use something else than a dot to store as property vs header, this is frankly a bit confusing.
Instead use a prefix such as property:, eg property:xxx

@jcszymansk
Copy link
Copy Markdown
Contributor Author

OK, updated.

@davsclaus davsclaus merged commit 483d7b8 into apache:camel-3.x May 15, 2023
@davsclaus
Copy link
Copy Markdown
Contributor

@jacekszymanski can you work on a PR for main branch so we will have this in Camel v4 as well.

@jcszymansk
Copy link
Copy Markdown
Contributor Author

Sure, I will, I should have it later today.

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.

3 participants