[ 
https://issues.apache.org/jira/browse/RATIS-459?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16792963#comment-16792963
 ] 

Tsz Wo Nicholas Sze commented on RATIS-459:
-------------------------------------------

Filed RATIS-499.

> Async requests may become out-of-order in some rare case
> --------------------------------------------------------
>
>                 Key: RATIS-459
>                 URL: https://issues.apache.org/jira/browse/RATIS-459
>             Project: Ratis
>          Issue Type: Improvement
>          Components: client, server
>            Reporter: Tsz Wo Nicholas Sze
>            Assignee: Tsz Wo Nicholas Sze
>            Priority: Major
>              Labels: ozone
>         Attachments: r459_20190222.patch, r459_20190227.patch, 
> r459_20190228.patch, r459_20190228b.patch, r459_20190308.patch, 
> r459_20190311.patch, r459_20190311b.patch
>
>
> With some discussion with [~ljain] (thanks!), the following (highly unlikely) 
> hypothetical scenario may lead to the async requests out-of-order.
> # Requests r1-r5 success, 
> # Request r6 fails with NotLeader, 
> # Request r7 fails with some other IOException
> # r6 is still keeping retrying and a retry of r7 succeeds with s1.
> Below are the suggested fix:
> - In SlidingWindow.Client, resetFirstSeqNum() should, but currently does not, 
> clear the request map.  
> - If a request is replied with NotLeaderException, it should not be removed 
> from the request map until resetFirstSeqNum() above.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to