PatrickRen opened a new pull request, #8679: URL: https://github.com/apache/paimon/pull/8679
Fix #8674 This pull request adds comprehensive integration tests for the S3 file system implementation, improves test infrastructure for S3 authentication scenarios, and simplifies the S3 plugin loader to better support credential-less authentication. The following are the most important changes: **Test Infrastructure and Coverage:** * Added `MinioTestContainer`, a reusable MinIO container test fixture to provide an S3-compatible backend for integration tests in `paimon-s3-impl`. This enables testing with both long-lived and temporary credentials. * Introduced `S3FileIOTest`, which covers S3 file system behavior using the MinIO test container, including tests for access key/secret key, session token authentication, and default credentials provider chain. * Updated `pom.xml` to include test dependencies on `paimon-common` and `paimon-test-utils`, enabling the new test infrastructure. **S3 Plugin Loader Improvements:** * Removed the requirement for access key and secret key options in `S3Loader`, allowing the S3 plugin to be used with credential-less authentication methods (such as IAM roles or environment variables). [[1]](diffhunk://#diff-9c4e07b0d8814f2183815075790837d6a02130eaf32c73f3fb64010193615536L28-L30) [[2]](diffhunk://#diff-9c4e07b0d8814f2183815075790837d6a02130eaf32c73f3fb64010193615536L57-L64) * Added `S3LoaderTest` to verify that `S3Loader` declares no required options and correctly reports its scheme as "s3". -- 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]
