slfan1989 commented on PR #1349: URL: https://github.com/apache/ratis/pull/1349#issuecomment-3911017568
> @slfan1989 , thanks a lot for working on this! > > it is a great idea to use exponential backoff. We use a similar idea for `raft.server.log.appender.retry.policy`. We have a `RetryPolicy` interface and a `MultipleLinearRandomRetry` implementation, which is very similar to Hadoop's implementation > > * https://github.com/apache/hadoop/blob/56192be95af06c0a04ae6503858199d240f823e0/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/retry/RetryPolicies.java#L387C23-L387C48 > > How about adding a new conf `raft.netty.dataStream.client.reconnect.policy` which uses `MultipleLinearRandomRetry` ? @szetszwo Thanks for the suggestion and for pointing out the existing retry policy! Yeah, I agree — since we already have `MultipleLinearRandomRetry` working well for log appender, reusing the same policy for `raft.netty.dataStream.client.reconnect.policy` makes a lot of sense. -- 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]
