+1 Yufei
On Tue, Mar 25, 2025 at 8:03 AM Prashant Singh <prashant.si...@snowflake.com.invalid> wrote: > +1 (non-binding) > > Best, > Prashant > > On Tue, Mar 25, 2025 at 7:01 AM Dmitri Bourlatchkov <di...@apache.org> > wrote: > > > +1 LGTM. > > > > Cheers, > > Dmitri. > > > > On Tue, Mar 25, 2025 at 12:30 AM Dennis Huo <huoi...@gmail.com> wrote: > > > > > Hello all, > > > > > > We've had some productive discussion in various places on the mailing > > list, > > > in the github PR, and in the live community sync now about the initial > > > minimal updates to the Polaris REST API (under the "management API") > for > > > adding a ConnectionConfigInfo field to ExternalCatalogs to represent > > > external federation: > > > > > > https://lists.apache.org/thread/szq8nzhblgwz0w71g5wh8kkcwbm7nhkn > > > https://github.com/apache/polaris/pull/1026 > > > > > > To recap, the the ConnectionConfigInfo is designed to be extensible so > > that > > > we can eventually define a variety of federation sources, but currently > > > just defines a minimal Iceberg REST federation so that we have a good > > > starting point to start implementing the internals of federation > without > > > biting off too much. > > > > > > We've also agreed the current proposed API is marked as "experimental" > > and > > > "subject to change", so this step also doesn't entrench the API > > definition > > > in stone yet. > > > > > > This thread commences our formal vote for addition of this initial API > > and > > > merging the PR listed above. > > > > > > [ ] +1 Merge https://github.com/apache/polaris/pull/1026 > > > [ ] +0 > > > [ ] -1 Do not merge because... > > > > > >
