kgusakov commented on code in PR #1644: URL: https://github.com/apache/ignite-3/pull/1644#discussion_r1102867316
########## modules/raft/tech-notes/src/primaryReplica.puml: ########## @@ -0,0 +1,34 @@ +@startuml primaryReplica +title PrimaryReplica and Replication Group communication (diagram 1) + +skinparam maxMessageSize 400 +skinparam defaultFontSize 12 + +participant PlacementDriver +participant PrimaryReplica + +participant Replica1 [ +Replica1 +Leader for term 1 +] + +participant Replica2 [ +Replica2 +Leader for term 2 +] + +PrimaryReplica -> Replica1 : Send a changePeersAsync request (node is the leader at the moment) +Replica1 -> Replica1 : Leader was stepped down and new leader election will start. +Replica2 -> Replica2 : Current node elected as a leader. +Replica2 -> PrimaryReplica : Send a message about the new leader elected. [see 3.1 details] +PrimaryReplica -> PrimaryReplica : Check local state for ongoing rebalance for the replica group. Review Comment: Yes, you are right. But fo this case it doesn't matter - will it be collocated or not, just for the explicitness of digram, I think we can leave it as is -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
