JingsongLi commented on PR #9903: URL: https://github.com/apache/paimon/pull/9903#issuecomment-5708351533
Retrying a transient DNS failure on the commit path is reasonable, and the cause-chain walk correctly handles wrapped exceptions. One note for the record: with the defaults (max-retries 5, initial delay 1 s, doubling), a genuinely dead host pins the commit thread for ~31 s (1+2+4+8+16) before failing. That is a reasonable trade-off and both knobs are configurable, but worth spelling out in the option docs so commit latency is not surprising. Also, this intentionally only covers `UnknownHostException` and not other transient connection errors — fine as a narrow scope, just worth documenting. -- 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]
