stn1slv opened a new pull request, #25321: URL: https://github.com/apache/camel/pull/25321
### What Updates the `camel-jms` documentation to reference the Jakarta Messaging API, and fixes several factual defects found in the same pages while doing so. **1. `javax.jms` -> `jakarta.jms` (23 references)** `camel-jms` has run on Jakarta Messaging since Camel 4. `components/camel-jms/src/main/java` contains 111 `jakarta.jms` imports and zero `javax.jms`, and `pom.xml` depends on `jakarta.jms:jakarta.jms-api` 3.1.0. The documentation, however, still described `javax.jms` throughout, including the normative type-mapping tables that readers copy type names from. Renamed 22 references in `jms-component.adoc` and 1 in `camelcontext.adoc`. Every type named (`Message`, `TextMessage`, `MapMessage`, `BytesMessage`, `ObjectMessage`, `Destination`) exists unchanged in `jakarta.jms-api` 3.1.0, so this is a pure package rename. The two `http://java.sun.com/.../javax/jms/Message.html` URLs are intentionally left as-is: they are historical links, not package references. **2. `preserveMessageQuo` -> `preserveMessageQos`** There is no option named `preserveMessageQuo`. The real option is `preserveMessageQos` (`JmsConfiguration.java`). A reader copying the documented name would get a silent no-op. The surrounding sentence also had an unbalanced parenthesis and a stale "from *Camel 2.6* onwards" qualifier, both cleaned up. **3. "Spring 2's `JmsTemplate`"** The build is on Spring 7. Changed to "Spring's `JmsTemplate`" so it does not go stale again. **4. Type-mapping tables were incomplete and unconditional** `JmsBinding.getJMSMessageTypeForBody` maps `File`, `Reader`, `InputStream`, `WrappedFile` and `StreamCache` to `StreamMessage` when `streamMessageTypeEnabled` is set, and forces `BytesMessage` when `artemisStreamingEnabled` is in effect. The tables stated the `BytesMessage` mapping unconditionally and did not mention `StreamMessage` at all, even on the receive side where `extractBodyFromJms` returns an `InputStream` for it. Added the missing `StreamMessage` receive row, the `WrappedFile` and `StreamCache` send rows, and a NOTE describing the `streamMessageTypeEnabled` / `artemisStreamingEnabled` behaviour. ### Why These are user-facing correctness problems rather than cosmetics. The package rename affects the tables people copy from, and `preserveMessageQuo` is an option name that does not exist. ### Testing Documentation-only change; no production code is modified. The regenerated catalog copy is included, as is the convention for component doc changes. --- _Generated by Claude Code on behalf of @stn1slv_ -- 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]
