runzhiwang commented on pull request #127: URL: https://github.com/apache/incubator-ratis/pull/127#issuecomment-647228570
@lokeshj1703 Thanks for review. > GrpcLogAppender should reset the client and retry the append entries. Are we seeing this behavior in the test? Do you mean if exception happens, client should reset the client and retry the append entries ? If so, I did not see this behavior in the test. And the reason of the problem of this PR, is [handleReply(reply)](https://github.com/apache/incubator-ratis/blob/master/ratis-grpc/src/main/java/org/apache/ratis/grpc/server/GrpcServerProtocolService.java#L125) processed by different thread of future. ---------------------------------------------------------------- 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]
