oscerd opened a new pull request, #23209:
URL: https://github.com/apache/camel/pull/23209

   ## Summary
   
   Fixes [CAMEL-23510](https://issues.apache.org/jira/browse/CAMEL-23510).
   
   The Exchange header constants in `JGroupsConstants` were using raw uppercase
   string values (`JGROUPS_DEST`, `JGROUPS_SRC`, `JGROUPS_CHANNEL_ADDRESS`,
   `JGROUPS_ORIGINAL_MESSAGE`) that do not follow the Camel naming convention
   (`CamelXxxYyy`) used across the rest of the component catalog.
   
   The Java field names are unchanged; only the header string values have
   changed:
   
   | Constant | Previous value | New value |
   | --- | --- | --- |
   | `JGroupsConstants.HEADER_JGROUPS_CHANNEL_ADDRESS` | 
`JGROUPS_CHANNEL_ADDRESS` | `CamelJGroupsChannelAddress` |
   | `JGroupsConstants.HEADER_JGROUPS_DEST` | `JGROUPS_DEST` | 
`CamelJGroupsDest` |
   | `JGroupsConstants.HEADER_JGROUPS_SRC` | `JGROUPS_SRC` | `CamelJGroupsSrc` |
   | `JGroupsConstants.HEADER_JGROUPS_ORIGINAL_MESSAGE` | 
`JGROUPS_ORIGINAL_MESSAGE` | `CamelJGroupsOriginalMessage` |
   
   Routes that reference the constants symbolically continue to work without
   changes. Routes that set the header by its literal string value must be
   updated to the new value (covered in the upgrade guide entry added in
   this PR).
   
   ## Changes
   
   - 
`components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsConstants.java`:
 rename the four header string values to the `CamelJGroups*` convention.
   - `docs/user-manual/modules/ROOT/pages/camel-4x-upgrade-guide-4_21.adoc`: 
add an upgrade-guide entry describing the rename.
   - Regenerated metadata: 
`components/camel-jgroups/src/generated/resources/META-INF/.../jgroups.json`, 
`catalog/camel-catalog/src/generated/resources/.../jgroups.json`, 
`dsl/camel-endpointdsl/src/generated/java/.../JGroupsEndpointBuilderFactory.java`.
   
   ## Test plan
   
   - [x] `mvn test` in `components/camel-jgroups` — all 18 tests pass.
   - [x] Full reactor `mvn clean install -DskipTests` — succeeds, no other 
modules require updates.
   
   ---
   
   _Claude Code on behalf of Andrea Cosentino_


-- 
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]

Reply via email to