[ 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)