The vote passes 7 +1 binding votes and 1 +1 non-binding vote and I will merge the spec change. Thanks everyone for providing feedback and voting!
Thanks, Amogh Jahagirdar On Wed, Aug 14, 2024 at 9:23 PM Renjie Liu <liurenjie2...@gmail.com> wrote: > +1 > > On Thu, Aug 15, 2024 at 10:10 AM Jack Ye <yezhao...@gmail.com> wrote: > >> +1 >> >> -Jack >> >> On Wed, Aug 14, 2024 at 8:39 AM Daniel Weeks <dwe...@apache.org> wrote: >> >>> +1 >>> >>> On Tue, Aug 13, 2024 at 7:34 PM xianjin <xian...@apache.org> wrote: >>> >>>> +1 >>>> >>>> On Aug 14, 2024, at 2:24 AM, Ryan Blue <b...@databricks.com.invalid> >>>> wrote: >>>> >>>> >>>> +1 >>>> >>>> On Tue, Aug 13, 2024 at 8:59 AM Yufei Gu <flyrain...@gmail.com> wrote: >>>> >>>>> +1 >>>>> Yufei >>>>> >>>>> >>>>> On Tue, Aug 13, 2024 at 8:57 AM Eduard Tudenhöfner < >>>>> etudenhoef...@apache.org> wrote: >>>>> >>>>>> +1 >>>>>> >>>>>> On Tue, Aug 13, 2024 at 5:09 PM Amogh Jahagirdar <2am...@gmail.com> >>>>>> wrote: >>>>>> >>>>>>> I've opened a PR [1] to clarify in the REST spec that if a server >>>>>>> receives an unknown update or requirement as part of any the commit >>>>>>> endpoints, that the server must fail with a 400 error response. >>>>>>> >>>>>>> Please vote on merging this change. The vote will remain open for >>>>>>> at least 72 hours. >>>>>>> >>>>>>> [] +1 >>>>>>> [] +0 >>>>>>> [] -1, do not merge because ... >>>>>>> >>>>>>> [1] https://github.com/apache/iceberg/pull/10848 >>>>>>> >>>>>>> Thanks, >>>>>>> >>>>>>> Amogh Jahagirdar >>>>>>> >>>>>> >>>> >>>> -- >>>> Ryan Blue >>>> Databricks >>>> >>>>
