devabhishekpal commented on code in PR #9867:
URL: https://github.com/apache/ozone/pull/9867#discussion_r2895669746
##########
hadoop-ozone/interface-client/src/main/proto/OmClientProtocol.proto:
##########
@@ -1726,6 +1730,19 @@ message PartKeyInfo {
required KeyInfo partKeyInfo = 3;
}
+message MultipartPartInfo {
+ optional string partName = 1;
+ optional uint32 partNumber = 2;
+ optional string eTag = 3;
+ optional KeyLocationList keyLocationList = 4;
+ optional uint64 dataSize = 5;
+ optional uint64 modificationTime = 6;
+ optional uint64 objectID = 7;
+ optional uint64 updateID = 8;
+ optional FileEncryptionInfoProto fileEncryptionInfo = 9;
+ optional FileChecksumProto fileChecksum = 10;
Review Comment:
I think per part owner metadata is not required.
This was mostly incidental metadata enforcement of ACL did not rely on
per-part owner. Also I think checks on native authorizer is done on bucket
level for listing operations. But maybe key write does open key ACL check for
that write.
I think owner metadata and acls can be pulled up to the MultipartInfo
message. Let me do that
--
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]
