[PR] CAMEL-24623: camel-infinispan - report a QUERY that has no query builder (backport to camel-4.18.x) [camel]

["oscerd (via GitHub)" <gi...@apache.org>]

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

   Backport of #26111 (CAMEL-24623) to `camel-4.18.x`. Clean cherry-pick, no conflicts.
   
   Both producers ran the `QUERY` operation through an `if (query != null)` with no else branch, so a route
   with neither the `queryBuilder` option nor the `CamelInfinispanQueryBuilder` header got its own message back
   with no result, no exception and no log line — indistinguishable from a query that matched nothing.
   
   The pass-through is deliberately **kept**: `producerQueryOperationWithoutQueryBuilder` has asserted it since
   the operation was contributed in CAMEL-9624 (2016). Only the silence goes away — both producers now log a
   WARN naming the cache, the option and the header, and the builder is resolved before the cache is looked up
   so a misconfigured remote route no longer pays a Hot Rod round trip to do nothing. The embedded producer
   carried the same defect and is fixed the same way.
   
   Safe for a patch release: log-only, no behaviour change, no metadata or catalog change.
   
   **Merge after #26111.**
   
   ### Verification
   
   `mvn clean install -DskipITs` on `components/camel-infinispan` is green on this branch — 89 embedded tests
   and the remote module's unit tests, including the 4 new ones in `InfinispanRemoteProducerQueryTest`.
   
   The full reactor was not run here: the host was under memory pressure and killed it twice. It would add
   nothing for this change — the module build regenerates nothing (`git status` shows only the five
   cherry-picked files), no annotation or metadata is touched, and the only module outside the infinispan
   family that depends on `camel-infinispan` is `camel-langchain4j-embeddings`, as a test dependency with no
   reference to the classes changed here.
   
   ---
   _Claude Code on behalf of oscerd_
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] CAMEL-24623: camel-infinispan - report a QUERY that has no query builder (backport to camel-4.18.x) [camel]

["davsclaus (via GitHub)" <gi...@apache.org>]

davsclaus merged PR #26117:
URL: https://github.com/apache/camel/pull/26117


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org