slfan1989 commented on PR #1349: URL: https://github.com/apache/ratis/pull/1349#issuecomment-3912088239
@szetszwo Thank you for the great suggestion! I took a closer look at `MultipleLinearRandomRetry` - it's indeed a well-designed implementation similar to Hadoop's approach. I noticed that Ratis already has an `ExponentialBackoffRetry` strategy. For this PR, would it be acceptable to implement `ExponentialBackoffRetry` first? We can use `reconnect.delay` / `reconnect.max-delay` (+ `reconnect.max-attempts`) to build the exponential backoff as the default policy. Then in a follow-up PR, I can add a configurable `raft.netty.dataStream.client.reconnect.policy` property to support multiple retry strategies (including `MultipleLinearRandomRetry`), similar to how `raft.server.log.appender.retry.policy` works. This way users will have the flexibility to choose different retry policies based on their needs. Does this approach sound reasonable? -- 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]
