oscerd commented on PR #25900: URL: https://github.com/apache/camel/pull/25900#issuecomment-5475493963
Thanks @davsclaus — fixed in the latest commit. `setProperties` now reads any existing `PROPERTIES` header and merges the text (and optional id) into a **copy** of it instead of building a fresh map, so caller-supplied properties survive. This restores `LangChain4jEmbeddingsComponentWeaviateTargetIT.querybyid` (the `sky`/`age`/`dog` entries set by `create()`/`updateById()` are preserved, with `text` added). I also extended `WeaviateEmbeddingsDataTypeTransformerTest` with a pre-populated-`PROPERTIES` case, which is exactly the scenario the original test missed. _Claude Code on behalf of oscerd_ -- 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]
