Skip to content

Conversation

@huangnauh
Copy link
Contributor

@huangnauh huangnauh commented Dec 31, 2021

Some Amazon S3 Compatible Storage only supports the same part size(1M-5G) in the multipart upload, expect for the last part. And the last part must not be larger than the same part size.

We add a option MultipartCombineSmallPart to enable/disable combining the pending part

Signed-off-by: Libo Huang huanglibo2010@gmail.com

@milosgajdos
Copy link
Member

hey @huangnauh thanks for the PR; can you please sign your commit as per the contributors guildeines?

@huangnauh
Copy link
Contributor Author

hey @huangnauh thanks for the PR; can you please sign your commit as per the contributors guildeines?

Done. PTAL

@codecov-commenter
Copy link

codecov-commenter commented Jan 4, 2022

Codecov Report

Merging #3556 (e107724) into main (6a977a5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3556   +/-   ##
=======================================
  Coverage   56.35%   56.35%           
=======================================
  Files         101      101           
  Lines        7307     7307           
=======================================
  Hits         4118     4118           
  Misses       2534     2534           
  Partials      655      655           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a977a5...e107724. Read the comment docs.

Signed-off-by: Libo Huang <huanglibo2010@gmail.com>
@milosgajdos milosgajdos merged commit c53f110 into distribution:main Jan 18, 2022
milosgajdos added a commit to milosgajdos/distribution that referenced this pull request Dec 18, 2023
This reverts distribution#3556

This feature is currently broken and requires more fundamental changes
in the S3 driver. Until then it's better to remove it.

Signed-off-by: Milos Gajdos <milosthegajdos@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants