Hi Dmitri, +1, personally I don't see any harm to add, plus besides supporting S3-compatible storages, this allows supporting legacy s3 buckets that still use path style access
Regards, Omar On Wed, Jul 9, 2025 at 1:52 AM Dmitri Bourlatchkov <di...@apache.org> wrote: > Hi All, > > I propose to add a boolean `pathStyleAccess` optional parameter > to AwsStorageConfigInfo in PR [2012]. > > The main idea is to support non-AWS S3 implementations for [1530] but the > new parameter is meaningful in AWS SDK S3 clients in general. > > If set, the value will be propagated to Iceberg REST Catalog clients under > the "s3.path-style-access" configuration property name. > > This change is backward-compatible with existing clients and deployed > catalogs. > > Thoughts? > > [1530] https://github.com/apache/polaris/issues/1530 > [2012] https://github.com/apache/polaris/pull/2012 >
