runzhiwang edited a comment on pull request #373: URL: https://github.com/apache/incubator-ratis/pull/373#issuecomment-750748956
> Sure. It should use max instead adding two timeout values together. @szetszwo Use the max does not work, because rpc call count time before TransferLeadership#scheduler, so if we use the max, such as 10 seconds, maybe when TransferLeadership#scheduler count time to 9.9 seconds, rpc call already count to 10 seconds, so rpc call timeout before TransferLeadership#scheduler timeout. So the timeout value of rpc call should be larger than TransferLeadership#schedule ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
