JingsongLi commented on PR #8148:
URL: https://github.com/apache/paimon/pull/8148#issuecomment-4642419795

   I found one issue with the OSS archive/unarchive implementation.
   
   `changeStorageClass` always uses a single `OSSClient.copyObject(...)` call 
to rewrite the object with a new storage class. Alibaba Cloud OSS documents 
`CopyObject` for objects up to 1 GB; larger objects should be copied with 
multipart copy / `UploadPartCopy` instead. Paimon data files can exceed 1 GB, 
so `archive` / `unarchive` may fail for large files.
   
   Could we branch on `sourceMetadata.getContentLength()` and use multipart 
copy for large objects, preserving the source metadata and setting the target 
storage class during the multipart upload? The current single-copy path can 
remain for objects within the supported size.
   
   Reference: 
https://www.alibabacloud.com/help/en/oss/developer-reference/copy-objects-8
   


-- 
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]

Reply via email to