davsclaus commented on PR #28210: URL: https://github.com/apache/camel/pull/28210#issuecomment-6325248416
Thanks @oscerd, the points hold. I merged it now so tonight's benchmark run has it. A follow-up PR will: 1. give the `resultType: java.util.Map` advice only when the index ends the path, and for `[0].field` say to leave the index out and set resultType to the field's type (in the eval note too); 2. in a `when` or `filter`, say that the filter alone is the condition, and add "null also when several match" to the one-element texts; 3. bring `noKeysHint` in line with the new list hint; 4. add tests: the note's wiring without the download gate, a `[0].qty` case, an `||` case, and a valid neighbour such as `$.orders[?(@.id == 1)].items[0]`; 5. use the imports in `CatalogDocsTest`. The NPE in `JsonPathExpression` (a Map result type on a list of values that are not maps, there since CAMEL-20508) gets its own Jira. _Claude Code on behalf of davsclaus_ -- 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]
