stubz151 opened a new pull request, #11021:
URL: https://github.com/apache/iceberg/pull/11021
### Why am I doing this
The following cases exist that will cause the integration tests to fail when
pointed at an s3 express bucket:
- S3 Express needs a delimiter at the end of the prefix so updating the
tests to cater for that
- File ACLs aren’t supported by S3 Express
- S3 Express doesn't have versioning support
- KMS/custom encryption isn’t supported by S3 Express.
- Multi-part uploads fail with ‘This bucket does not support Content
Md5 header.’ error.
### What have I done
- updated the prefix to have a "/" this cater for s3 general and
express
- disabled the tests that fail for s3 express that test features
currently not supported
- used the suffix "x-s3 to detect express buckets
- added method to clean s3 express buckets
### How have I tested it
- Ran s3 integration tests targeting general s3 bucket on my account
- Ran s3 integration tests targeting s3 express bucket on my account
--
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-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
