sodonnel commented on a change in pull request #2401:
URL: https://github.com/apache/ozone/pull/2401#discussion_r677606724
##########
File path: hadoop-ozone/interface-client/src/main/proto/OmClientProtocol.proto
##########
@@ -519,6 +519,7 @@ message BucketInfo {
optional int64 quotaInBytes = 15 [default = -2];
optional int64 quotaInNamespace = 16 [default = -2];
optional uint64 usedNamespace = 17;
+ optional hadoop.hdds.ECReplicationConfig ecReplicationConfig = 18;
Review comment:
Discussed with Uma, and we think the best way forward is to create a new
structure called:
```
defaultReplicationConfig {
optional hadoop.hdds.Type type = 1;
optional hadoop.hdds.Factor factor = 2;
optional hadoop.hdds.ECReplicationConfig ecReplicationConfig = 3;
}
```
While we do not have a backwards compatibility issue here, it keeps it
consistent with the other parts of the code was have change, and we can reuse
the same logic to create the internal classes from the proto structures.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
