+1 (binding, also approved in GH) Thanks for the CHANGELOG update, Adnan!
Given that the purpose of this vote is to confirm the REST API response changes, I believe the CHANGELOG update does _not_ invalidate previously cast votes. Cheers, Dmitri. On Fri, Sep 19, 2025 at 6:41 PM Adnan Hemani <[email protected]> wrote: > Thanks all - I’ve pushed the change to the CHANGELOG. Dmitri, please take > a look and let us know if that changes your vote! > > Best, > Adnan Hemani > > > On Sep 19, 2025, at 3:20 PM, Honah J. <[email protected]> wrote: > > > > +1 (non-binding) and +1 for adding that to the changelog. > > > > Thanks Adnan for working on this! > > > > Best regards, > > Jonas > > > > > > On Fri, Sep 19, 2025 at 5:19 PM Michael Collado <[email protected] > <mailto:[email protected]>> > > wrote: > > > >> +1 > >> > >> Mike > >> > >> On Fri, Sep 19, 2025 at 10:46 AM Alexandre Dutra <[email protected]> > >> wrote: > >> > >>> +1 for the change and +1 for the changelog. Thanks Adnan! > >>> > >>> On Fri, Sep 19, 2025 at 7:39 PM Yufei Gu <[email protected]> wrote: > >>>> > >>>> +1 Thanks for working on this, Adnan! It's also nice to mention it in > >> the > >>>> changelog as Dmitri pointed out. > >>>> > >>>> Yufei > >>>> > >>>> > >>>> On Fri, Sep 19, 2025 at 8:58 AM Dmitri Bourlatchkov > >>>> <[email protected]> wrote: > >>>> > >>>>> +0 As I commented in GH, code changes LGTM (as of commit f0b6896), > >> but > >>> I > >>>>> believe we ought to mention the change in API behaviour in > >> CHANGELOG.md > >>>>> > >>>>> Cheers, > >>>>> Dmitri. > >>>>> > >>>>> On Thu, Sep 18, 2025 at 9:46 PM Adnan Hemani > >>>>> <[email protected]> wrote: > >>>>> > >>>>>> Hi all, > >>>>>> > >>>>>> I propose that we make a change to the OpenAPI spec to return the > >>> newly > >>>>>> created object in each of the following APIs: createCatalog, > >>>>>> createPrincipalRole, and createCatalogRole. > >>>>>> > >>>>>> ML thread regarding this topic here < > >>>>>> > https://www.google.com/url?q=https://lists.apache.org/thread/q7q0rrsmw5gcqv30g4hr9ffq3gtr72yk&source=gmail-imap&ust=1758925248000000&usg=AOvVaw1_MZoeQK5_2dJs_AoVYt9r > >. > >> A > >>>>>> draft PR detailing these changes can be found here < > >>>>>> > https://www.google.com/url?q=https://github.com/apache/polaris/pull/2603&source=gmail-imap&ust=1758925248000000&usg=AOvVaw27OBs3_oKo5tryrObBYq66 > >. > >>>>>> > >>>>>> Please vote in the next 72 hours. > >>>>>> > >>>>>> [ ] +1 Make the proposed API change. > >>>>>> [ ] +0 > >>>>>> [ ] -1 Do not make this change because… > >>>>>> > >>>>>> Only PPMC members and mentors have binding votes, but other > >> community > >>>>>> members are encouraged to cast non-binding votes. This vote will > >>> pass if > >>>>>> there are 3 binding +1 votes and more binding +1 votes than -1 > >> votes. > >>>>>> > >>>>>> Best, > >>>>>> Adnan Hemani > >
