xtern commented on code in PR #4256:
URL: https://github.com/apache/ignite-3/pull/4256#discussion_r1758317160


##########
modules/raft/src/main/java/org/apache/ignite/internal/raft/server/impl/JraftServerImpl.java:
##########
@@ -856,4 +860,54 @@ public void onLeaderStart(long term) {
             listener.onLeaderStart();
         }
     }
+
+    /** Returns raft group state accessor. */
+    public <G extends ReplicationGroupId, L> RaftGroupStateProvider<G, L> 
getGroupStateProvider(

Review Comment:
   @sanpwc we need to collect timestamps from ALL replication groups (primary 
and non-primary), can you clarify is this possible to do with proposed approach?
   Can you briefly describe how to do this?
   
   Currently we send a request to PartitionReplicaListener and apply the raft 
command, and as far as I remember we don't even know when the command will be 
applied to ALL replicas in the group (we only know when it will be applied to 
the primary replica) :thinking: .



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

Reply via email to