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

Lokesh Jain commented on RATIS-678:
-----------------------------------

[~swagle] Thanks for working on this! The changes look good to me. I think we 
can call notifyLeader for follower as well. Whenever a server changes to 
follower state, StateMachine#notifyLeader can be called to inform the server 
state machine about leader election. We can also include the leaderId in the 
function argument. This information can be used in ozone to inform about leader 
change for a pipeline. We can change the api name accordingly.

> Notify Leader does not provide raft group id
> --------------------------------------------
>
>                 Key: RATIS-678
>                 URL: https://issues.apache.org/jira/browse/RATIS-678
>             Project: Ratis
>          Issue Type: Improvement
>          Components: raft-group
>    Affects Versions: 0.4.0
>            Reporter: Siddharth Wagle
>            Assignee: Siddharth Wagle
>            Priority: Major
>             Fix For: 0.5.0
>
>         Attachments: RATIS-678.01.patch
>
>
> org.apache.ratis.statemachine.StateMachine#notifyLeader
> does not provide the group id for which leader election is complete.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to