jojochuang commented on code in PR #7109:
URL: https://github.com/apache/ozone/pull/7109#discussion_r2023720736
##########
hadoop-ozone/interface-client/src/main/proto/OmClientProtocol.proto:
##########
@@ -754,7 +754,7 @@ message BucketInfo {
required string bucketName = 2;
repeated OzoneAclInfo acls = 3;
required bool isVersionEnabled = 4 [default = false];
- required StorageTypeProto storageType = 5 [default = DISK];
+ optional StorageTypeProto storageType = 5 [default = DISK, deprecated =
true];
Review Comment:
is this change from required to optional backward compatible?
Imagine a new OM server and an old client. The new server omits this field
while client expects this field. Wouldn't the client fail to parse?
--
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]
