davsclaus commented on PR #26917: URL: https://github.com/apache/camel/pull/26917#issuecomment-5848951898
Correcting the evidence, though the improvement itself stands. I said the model called `camel_catalog_doc(sql)` six times because it could not find the named-parameter syntax. Reading the traces properly, it was asking about something else: *"let me check how to handle upsert in H2"*, *"check the SQL component for how to handle upserts and named parameters"*. The parameter syntax it had already been given by the validator (CAMEL-24869) when `camel_edit_file` refused its write. What it was actually hunting for was H2's upsert dialect, which no Camel catalog can supply. So the excerpt would not have rescued that step either. The gap this issue describes is still real and is worth fixing on its own terms: an option list cannot teach syntax that is not an option, and for `sql` the `:#name` form appears nowhere in 15 KB of options while the component's page explains it in its first 80 lines. The excerpt puts it in the answer. I have simply removed the benchmark step from the case for it. -- 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]
