Re: [PR] fix!(io): bound S3 write request size with a configurable part size [iceberg-rust]

["YuangGao (via GitHub)" <gi...@apache.org>]

YuangGao commented on PR #3179:
URL: https://github.com/apache/iceberg-rust/pull/3179#issuecomment-5726007750

   thanks for the review!  I have pushed the fixes. The `write()` question turned out the other way round — details in that thread, short version: `write_options()` does chunk, and small metadata writes don't pick up multipart. Added two MinIO tests pinning both. Ceiling, const, error units, imports and the ETag note are all in. Two small deviations from the suggestions, noted inline.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org