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

Chen Liang commented on RATIS-141:
----------------------------------

+1 on v20171125b patch, I've committed the patch, thanks [~szetszwo] for the 
contribution!

> In RaftClientProtocolService, the assumption of consecutive callId is invalid
> -----------------------------------------------------------------------------
>
>                 Key: RATIS-141
>                 URL: https://issues.apache.org/jira/browse/RATIS-141
>             Project: Ratis
>          Issue Type: Bug
>            Reporter: Tsz Wo Nicholas Sze
>            Assignee: Tsz Wo Nicholas Sze
>         Attachments: r141_20171117.patch, r141_20171119b.patch, 
> r141_20171120.patch, r141_20171124.patch, r141_20171125.patch, 
> r141_20171125b.patch
>
>
> {code}
> //RaftClientProtocolService.AppendRequestStreamObserver.onNext(..)
>               // we assume the callId is consecutive for a stream RPC call
>               final PendingAppend pendingForReply = pendingList.get(
>                   (int) (replySeq - headSeqNum));
> {code}
> Call id is used for different kinds of calls (e.g. getInfo) so that it may 
> not be consecutive.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to