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

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

Question: do we need only the groupId?  Or also the group membesr?

In StateMachine.initialize, the groupId is passed to the StateMachine so that 
the StateMachine can remember it.  If groupId is added to 
notifyInstallSnapshotFromLeader, it gives an impression that it may be 
different from the groupId passed to StateMachine.initialize.

> Add raft group Id to install snapshot notification
> --------------------------------------------------
>
>                 Key: RATIS-564
>                 URL: https://issues.apache.org/jira/browse/RATIS-564
>             Project: Ratis
>          Issue Type: Improvement
>          Components: server
>    Affects Versions: 0.3.0
>            Reporter: Siddharth Wagle
>            Assignee: Siddharth Wagle
>            Priority: Major
>             Fix For: 0.4.0
>
>         Attachments: RATIS-564.01.patch
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The current API: 
> org.apache.ratis.statemachine.StateMachine#notifyInstallSnapshotFromLeader, 
> only provides the term index and does not provide the raft group information 
> to the follower.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to