carlesarnal commented on PR #24844:
URL: https://github.com/apache/camel/pull/24844#issuecomment-6033014886

   Thanks both. I pushed 9ef714c5d5b and replied on each inline thread.
   
   @davsclaus, for the points outside the inline threads:
   - **A version that always fails:** the docs now say it blocks newer versions 
and show how to skip it with `onException(...).handled(true)`. There is a route 
test where version 1 always fails and is handled, and version 2 is still 
delivered.
   - **Logged twice:** when the route's error handler has already dealt with 
the failure, the consumer now only logs at debug. It still uses the exception 
handler when no error handler was involved.
   - **Deletion between pages:** documented as a known limitation of offset 
paging.
   - **Redundant watermark check:** removed.
   
   @oscerd, all of your points are addressed: DISABLED and missing versions 
with fetchContent, searchArtifacts filtering and paging, `security = "secret"`, 
fail-fast ID checks, the idle-poll probe, TLS/proxy options, and docs for 
deletion and consumer headers.
   
   Locally, 53 unit tests and 8 integration tests pass. The generated catalog 
and DSL files are updated.
   
   _AI-assisted response prepared with OpenCode on behalf of @carlesarnal._


-- 
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]

Reply via email to