[jira] [Comment Edited] (RATIS-100) Test multiple raft groups with a state machine

["Tsz Wo Nicholas Sze (JIRA)" <ji...@apache.org>]

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

Tsz Wo Nicholas Sze edited comment on RATIS-100 at 8/4/17 8:36 PM:
-------------------------------------------------------------------

r100_20170804.patch: adds TestMultiRaftGroup.

Note that the new test may fail due to some existing bug that a RaftServer may stay in STARTING state fever.  Will fix the bug and upload a new patch later.

BTW, this fixes an existing bug that PendingRequests.addPendingRequest(..) may throw IllegalStateException when addPendingRequest(..) is called right after addConfRequest(..) since the index in last PendingRequest does not match.  This patch fixes the bug by setting {{last = pendingSetConf}} in addConfRequest(..).


was (Author: szetszwo):
r100_20170804.patch: adds TestMultiRaftGroup.

Note that the new test may fail due to some existing bug that a RaftServer may stay in STARTING state fever.  Will fix the bug and upload a new patch later.

BTW, this fixes an existing bug that PendingRequests.addPendingRequest(..) may throw IllegalStateException when addPendingRequest(..) is called right after addConfRequest(..) since the index in last PendingRequest does not match.  This patch fixes the buy by setting {{last = pendingSetConf}} in addConfRequest(..).

> Test multiple raft groups with a state machine
> ----------------------------------------------
>
>                 Key: RATIS-100
>                 URL: https://issues.apache.org/jira/browse/RATIS-100
>             Project: Ratis
>          Issue Type: Test
>            Reporter: Tsz Wo Nicholas Sze
>            Assignee: Tsz Wo Nicholas Sze
>         Attachments: r100_20170804.patch
>
>
> We propose to add a test similar to ReinitializationBaseTest.runTestReinitializeMultiGroups(..) with a state machine so that it can test if the states are recorded correctly.



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