ramu11 commented on PR #27314: URL: https://github.com/apache/camel/pull/27314#issuecomment-6042297724
Thanks @davsclaus for the detailed review. I’ve addressed the latest review comments and pushed the updated branch: - Updated `firstVersion` to **4.24.0** in the POM and `@UriEndpoint`. - Rebased onto the latest `upstream/main` and regenerated the catalog/DSLs, including the updated `copyParameters(properties)` generation. - Strengthened the JTA test to verify transaction begin/commit, no rollback, session close, result body, and the transaction coordinator lookup. - Updated `naturalIdParameters` and filters with URI prefixes and multi-value support, including message-based natural-id values, while retaining Java / `#bean` usage. - Cached the JTA transaction-coordinator detection during endpoint startup instead of resolving it for every exchange. - Added/updated the corresponding tests and documentation. The `camel-hibernate` test suite passes (**48 tests, 0 failures**) and `git diff --check` is clean. The Hibernate 8 Beta / Jakarta Persistence milestone dependency remains the intentional release blocker. I’m keeping the PR scoped to Hibernate 8 and will wait for the GA dependencies rather than moving it to Hibernate 7.4.x. AI assistance: Assisted by Grok-4.6 (Cursor) for chat assistance and review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
