+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... > > > > > > > > > >
