slbotbm commented on code in PR #3729: URL: https://github.com/apache/iggy/pull/3729#discussion_r3839806735
########## core/integration/src/harness/handle/connectors_runtime.rs: ########## Review Comment: `DEFAULT_HEALTH_CHECK_RETRIES` is 1,000 and the interval is 20 ms. On the black-holed request case these serial 2-second client timeouts make `wait_ready()` take roughly 34 minutes, so the harness can still hang for a long time even after this change. Bound the whole readiness loop with a deadline, or reduce the retry budget to account for the per-request timeout, rather than only bounding each probe. -- 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]
