symious commented on PR #989: URL: https://github.com/apache/ratis/pull/989#issuecomment-1854999331
> repeating reset. https://github.com/apache/ratis/blob/cedcd2ad4cd2da13230aaa0d15678e5aee9b0729/ratis-client/src/main/java/org/apache/ratis/client/impl/OrderedAsync.java#L257C61-L257C61 Steps: 1. Client send 100 async requests to server, here firstReplied is true 2. Leader changed, 100 requests replied with exceptions, trying to reset the slidingWindow. 3. Say the requests id are [100, 199], client receives "id100" and and reset the slidingWindow, firstReplied changed to false. 4. Client retry on "id100" and receives the reply, so set firstReplied to true. 5. Client then receives other requests' exception reply from Step2, then reset the slidingWindow again, but the connection is ok now, no need to reset again. @szetszwo -- 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]
