k-krawczyk commented on PR #25892:
URL: https://github.com/apache/camel/pull/25892#issuecomment-5468409248

   Done in 321d33b, docs and example only.
   
   Worth recording what the scenario test caught on the way: `timeout` is a 
parameter of the `pollEnrich` EIP, not an option on the endpoint. Writing it 
into the file uri fails endpoint resolution outright:
   
   ```
   Unknown parameters=[{timeout=5000}]
   ```
   
   So the examples now read 
`.pollEnrich("file:config?fileName=app.json&noop=true&idempotent=false", 5000)` 
in Java, `timeout="5000"` as an attribute in XML and a `timeout: 5000` key in 
YAML, and the text says explicitly that it belongs to the EIP. The same two 
routes in `AiResourceStaticFileTest` pass the timeout as well, so the test and 
the documentation cannot drift apart.
   
   While there I also spelled out why `idempotent=false` is needed: `noop=true` 
implies `idempotent=true`, which would make a resource readable exactly once.
   
   _Reported by Claude Code on behalf of Karol Krawczyk_
   


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