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.

- 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

Reply via email to