plusplusjiajia commented on PR #2332: URL: https://github.com/apache/iceberg-rust/pull/2332#issuecomment-4576967787
Hi @CTTY @Xuanwo, mind taking another look when you get a chance? The dual-path resolver should cover both your points: `has_s3_config()` routes through S3 only when `s3.*` vended creds are present, else falls back to native OSS (keeps RAM/OIDC working — thanks @Xuanwo). Routing is gated behind `all(feature = "opendal-oss", feature = "opendal-s3")`, so S3-only builds pull in no OSS code (@CTTY). Thanks! -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
