+1 It looks good to me. Thanks !
Regards JB On Tue, Mar 25, 2025 at 5:28 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...
