slfan1989 commented on PR #14243: URL: https://github.com/apache/iceberg/pull/14243#issuecomment-3377058070
> @pvary @mxm A minor enhancement that improves ZkLockFactory by adding support for all native ZooKeeper retry policies. > > Could you please help review this PR? Thank you very much! @pvary When you have some time, Could you please help review this PR? Thank you very much! The main purpose of this change is to enhance the ZooKeeper lock factory (ZkLockFactory) by introducing configurable retry policies. Here’s a brief summary of the newly added retry mechanisms: - `ONE_TIME` — Retries only once after the first failure; a simple and fast-failing strategy. - `N_TIME` — Keeps retrying until the maximum number of retries is reached. - `BOUNDED_EXPONENTIAL_BACKOFF` — Similar to exponential backoff, but capped by a maximum sleep time (maxSleepTimeMs). - `UNTIL_ELAPSED` — Keeps retrying until a specified total elapsed time limit is reached, then stops. These policies make the retry behavior of ZkLockFactory more flexible and adaptable across different deployment environments, while maintaining backward compatibility with the existing implementation. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
