With 7 +1 votes (5 binding) this passes. Thanks everyone, I'll get the PR merged.
On Tue, Nov 18, 2025 at 3:02 AM Prashant Singh <[email protected]> wrote: > +1 (non-binding) > > Best, > Prashant > > On Tue, Nov 18, 2025 at 12:50 AM Daniel Weeks <[email protected]> wrote: > >> +1 >> >> On Mon, Nov 17, 2025 at 11:17 AM Steven Wu <[email protected]> wrote: >> >>> +1 >>> >>> On Mon, Nov 17, 2025 at 11:13 AM huaxin gao <[email protected]> >>> wrote: >>> >>>> +1 (non-binding) >>>> >>>> On Mon, Nov 17, 2025 at 11:03 AM Yufei Gu <[email protected]> wrote: >>>> >>>>> +1 >>>>> Yufei >>>>> >>>>> >>>>> On Mon, Nov 17, 2025 at 10:54 AM Amogh Jahagirdar <[email protected]> >>>>> wrote: >>>>> >>>>>> +1 >>>>>> >>>>>> On Mon, Nov 17, 2025 at 11:48 AM Ryan Blue <[email protected]> wrote: >>>>>> >>>>>>> +1 >>>>>>> >>>>>>> This looks good to me. >>>>>>> >>>>>>> On Mon, Nov 17, 2025 at 12:48 AM Eduard Tudenhöfner < >>>>>>> [email protected]> wrote: >>>>>>> >>>>>>>> Hey everyone, >>>>>>>> >>>>>>>> For server-side scan planning we missed adding storage credentials, >>>>>>>> hence I'm proposing to add them to the response of the */plan* >>>>>>>> endpoint. >>>>>>>> >>>>>>>> The OpenAPI changes can be seen in PR #14563 >>>>>>>> <https://github.com/apache/iceberg/pull/14563>. >>>>>>>> >>>>>>>> Looking forward to your thoughts and feedback. >>>>>>>> >>>>>>>> Thanks, >>>>>>>> Eduard >>>>>>>> >>>>>>>
