[jira] [Comment Edited] (RATIS-564) Add raft group Id to install snapshot notification

["Siddharth Wagle (JIRA)" <ji...@apache.org>]

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

Siddharth Wagle edited comment on RATIS-564 at 5/30/19 6:14 PM:
----------------------------------------------------------------

[~szetszwo] The other methods *notifyNotLeader* and *notifySlowness*, get the groupInfo and roleInfoProto in their method signature, hence I made this change. But I am ok to drop the patch, just need to make sure we are on the same page. Thanks. Actually, how do I get *roleInfoProto* on the statemachine side?


was (Author: swagle):
[~szetszwo] The other methods *notifyNotLeader* and *notifySlowness*, get the groupInfo and roleInfoProto in their method signature, hence I made this change. But I am ok to drop the patch, just need to make sure we are on the same page. Thanks.

> 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)