Croway commented on PR #27279: URL: https://github.com/apache/camel/pull/27279#issuecomment-5952775326
Thanks @davsclaus, all addressed in 3a5caeffdaa2: 1. Regression test `keepsTheConnectionWhileASlowStatusIsCollected`: a status that takes 2 s, a 200 ms heartbeat; `hello` must arrive within 1.5 s and the client must stay on its first connection for 6 s. Without the fix it fails (`hello` after 2.2 s, reconnect after about 6 s). 2. Comments updated (inline reply). 3. Backpressure: the next round waits for the previous round's frames to be sent (inline reply). 4. Debug snapshot: added a comment that it shares the collector with the status (not a regression, as you say). 5. PR description now notes that the first status no longer comes with `hello`. Agreed on the file transport: same root cause without the heartbeat symptom, best fixed at the source by CAMEL-25267. _Claude Code on behalf of Croway_ -- 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]
