Hi Phillip, Thanks for bringing this topic to the community's attention!
I reviewed the PR and am +1 with the overall direction taken there. Thanks, Alex On Wed, Feb 11, 2026 at 11:29 PM Yufei Gu <[email protected]> wrote: > > Thanks Phillip for raising it. It does seem a right direction given the > discussion in the upstream PR, https://github.com/apache/iceberg/pull/13212. > I will need to take a close look. > > Yufei > > > On Wed, Feb 11, 2026 at 7:20 AM Dmitri Bourlatchkov <[email protected]> > wrote: > > > Hi Phillip, > > > > Thanks for starting this discussion! > > > > From my POV the REST API change is backward-compatible and a nice > > enhancement to Polaris. > > > > I posted some technical review comments in GH. > > > > It would be nice to get additional reviews from people more familiar with > > the External Catalogs feature. > > > > Cheers, > > Dmitri. > > > > On Wed, Feb 11, 2026 at 6:12 AM Phillip Henry <[email protected]> > > wrote: > > > > > I've added a REST API class (GcpAuthenticationParameters) in an attempt > > to > > > address #3451 <https://github.com/apache/polaris/issues/3451> - that is, > > > allowing users to create external catalogs that use the GoogleAuthManager > > > in the Iceberg codebase. > > > > > > I've put together this pull request > > > <https://github.com/apache/polaris/pull/3729> that hopefully implements > > > this. Other than adding an extra enum element to > > AuthenticationParameters, > > > there are no changes to existing classes. > > > > > > Dmitri Bourlatchkov has told me this necessitates a discussion in the > > > mailing lists and asked me to kick one off. So can I please ask for > > > opinions? > > > > > > Thanks, > > > > > > Phillip > > > > >
