+1 Best, Yun
On Tue, Mar 25, 2025 at 9:22 AM Honah J. <hon...@apache.org> wrote: > +1 (non-binding) > > Best, > Honah (Jonas) > > On Tue, Mar 25, 2025 at 8:22 AM Yufei Gu <flyrain...@gmail.com> wrote: > > > +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... > > > > > > > > > > > > > > >
