[ 
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/9/17 12:31 AM:
--------------------------------------------------------------------

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 forever.  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 bug 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)

Reply via email to