Re: Catalog Federation API Spec updates - adding ConnectionConfigInfo

2025-03-20 Thread Dmitri Bourlatchkov
Thanks for pushing this forward, Dennis! For visibility: I approved the PR on GitHub. If anyone has any comments or concerns that we did not cover so far, please reply here or on the PR. As a follow-up, I guess we need a separate discussion about handling secrets in Polaris in general, which wil

Re: Catalog Federation API Spec updates - adding ConnectionConfigInfo

2025-03-20 Thread Dennis Huo
Thanks for the comments! Yes I agree we'll probably want to adjust the API, and we can definitely try to start with some reasonable placeholder around auth/secrets to unblock various parallel streams of work on federation while keeping an eye on how different secrets/auth models can be accommodated

Re: Catalog Federation API Spec updates - adding ConnectionConfigInfo

2025-03-19 Thread Dmitri Bourlatchkov
Thanks, Dennis, for starting this discussion. As for me the PR looks ok in its current state, although I posted a few comments. I tend to think that in order to proceed with the federated catalogs implementation, it is probably best to merge the PR now and later rework the parts that deal with ex