elek commented on a change in pull request #1973:
URL: https://github.com/apache/ozone/pull/1973#discussion_r590405743
##########
File path:
hadoop-hdds/interface-server/src/main/proto/ScmServerDatanodeHeartbeatProtocol.proto
##########
@@ -206,6 +206,10 @@ message ContainerReplicaProto {
optional int64 deleteTransactionId = 11;
optional uint64 blockCommitSequenceId = 12;
optional string originNodeId = 13;
+ //the index of this replica in the replication group.
+ //this will be different for each instance in case of EC
+ //but the same (0) for all instances for standard Ratis
+ optional int32 replicationIndex = 14;
Review comment:
I prefer to post smaller patches (when possible) to make it easier to
review.
But in this case it's a fair question: it can be harder to imagine how would
it be used from SCM protocol, so I quickly created the next patch on top of it:
please see pr #2010
(Note: even #2010 not a full refactor we need more patches to refactor the
pipeline management... Or, it can be part of HDDS-4892)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
