hanishakoneru commented on pull request #2813:
URL: https://github.com/apache/ozone/pull/2813#issuecomment-1062463850
From what I understand, versioning is not supported in Multipart keys
currently.
In _S3MultipartUploadCompleteRequest#getOmKeyInfo()_:
```
// Already a version exists, so we should add it as a new version.
// But now as versioning is not supported, just following the commit
// key approach. When versioning support comes, then we can uncomment
// below code keyInfo.addNewVersion(locations);
// As right now versioning is not supported, we can set encryption info
// at KeyInfo level, but once we start supporting versioning,
// encryption info needs to be set at KeyLocation level, as each
version
// will have it's own file encryption info.
```
As such, the bug reported in HDDS-6261 should not effect Multipart keys.
cc. @smengcl @errose28
@kuenishi I will give it a day before committing in case Siyao or Ethan have
any other comments. Thanks.
--
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]
