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

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

> ... a server should not response to another peer if it is not in current 
> conf. ...

Instead of using conf, it is better to check if they are in the same group.

r100_20170809c.patch:
- In RaftServerImpl.appendEntries, check if the leader and the server are in 
the same group.
- In LeaderState.checkNewPeers(),
-* fails stagingState only if #no-progress-servers > sender.size()/2;
-* applyOldNewConf if caughtUp + attendVote > senders.size()/2.

> 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, r100_20170809b.patch, 
> r100_20170809c.patch, r100_20170809.patch, r100_no_leader_case.log
>
>
> 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