Thanks for working on this. Makes sense to add `pathStyleAccess`. Yufei
On Wed, Jul 9, 2025 at 12:34 AM Omar Al-Safi <oals...@apache.org> wrote: > 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 > > >
