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