oscerd opened a new pull request, #23442:
URL: https://github.com/apache/camel/pull/23442
## Summary
Renames the Exchange header string values in `ElasticsearchConstants` and
`OpensearchConstants` from non-Camel prefixed values (`operation`, `indexId`,
`indexName`, `documentClass`, `waitForActiveShards`, `scrollKeepAliveMs`,
`useScroll`, `size`, `from`, `enableDocumentOnlyMode`) to
`CamelElasticsearch*` / `CamelOpensearch*`, following the convention
used across the rest of the Camel component catalog and matching the
pattern established in CAMEL-23526 (`camel-cxf`), CAMEL-23522 (`camel-mail`),
CAMEL-23461 (`camel-aws-bedrock`), CAMEL-23532 (`camel-vertx-websocket` /
`camel-atmosphere-websocket` / `camel-iggy`), and CAMEL-23576 (`camel-jira`).
The Java field names (`PARAM_OPERATION`, `PARAM_INDEX_ID`,
`PARAM_INDEX_NAME`,
`PARAM_DOCUMENT_CLASS`, `PARAM_WAIT_FOR_ACTIVE_SHARDS`,
`PARAM_SCROLL_KEEP_ALIVE_MS`, `PARAM_SCROLL`, `PARAM_SIZE`, `PARAM_FROM`,
`PARAM_DOCUMENT_MODE`) are unchanged so routes referencing the constants
symbolically continue to work; routes using the literal string values
must be updated (documented in the 4.21 upgrade guide).
## Notes
- The already-`Camel`-prefixed `PROPERTY_SCROLL_ES_QUERY_COUNT` and
`PROPERTY_SCROLL_OPENSEARCH_QUERY_COUNT` constants are unchanged.
- The two components are paired in this single PR because they share an
identical header surface (originally one was forked from the other) and
the upgrade-guide entry covers them together.
## Generated artifacts
- `components/camel-elasticsearch/.../elasticsearch.json` + catalog mirror
- `components/camel-opensearch/.../opensearch.json` + catalog mirror
- `dsl/camel-endpointdsl/.../ElasticsearchEndpointBuilderFactory.java`
(`operation()` → `elasticsearchOperation()`, etc.)
- `dsl/camel-endpointdsl/.../OpensearchEndpointBuilderFactory.java`
(`operation()` → `opensearchOperation()`, etc.)
## Backports
Both components exist on `camel-4.18.x` and `camel-4.14.x` with the same
constant values. Backports will be filed as follow-up PRs once this lands.
## Test plan
- [x] `mvn test` in `components/camel-elasticsearch` — passes (unit tests
have no body; the integration tests live in `*IT.java` files and
require a running cluster)
- [x] `mvn test` in `components/camel-opensearch` — passes
- [x] Module + catalog + endpointdsl built cleanly; only es/os regen
artifacts in `git status`
- [x] DSL accessor names verified to follow `elasticsearch<Name>()` /
`opensearch<Name>()` convention
- [x] Upgrade guide entry added under `=== camel-elasticsearch /
camel-opensearch`
Tracker: CAMEL-23577
_Reported by 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]
