The vote passes with: 5 "+1 Binding votes" 3 "+1 Non-binding votes." 0 "-1 votes"
Actions to be taken: 1. Update the language/location of the clarification on time travel in https://github.com/apache/iceberg/pull/8982 and then have a committer/PMC member merge. I'll try to have this updated by Monday. 2. Merge https://github.com/apache/iceberg/pull/8981 (it seems there is no further feedback on this). Thanks everyone for the feedback. -Micah On Fri, Jul 19, 2024 at 9:03 AM Jack Ye <[email protected]> wrote: > +1 (binding) > > added minor comments to the time travel PR. > > Best, > Jack Ye > > On Fri, Jul 19, 2024 at 8:22 AM Daniel Weeks <[email protected]> wrote: > >> +1 (binding) >> >> Thanks, Micah. >> >> On Thu, Jul 18, 2024 at 8:29 PM Amogh Jahagirdar <[email protected]> >> wrote: >> >>> +1 (non-binding) on these spec clarifications >>> >>> Thanks, >>> Amogh Jahagirdar >>> >>> On Thu, Jul 18, 2024 at 5:08 PM Steven Wu <[email protected]> wrote: >>> >>>> I am +1 for the spec clarifications. >>>> >>>> I have left some comments for the time travel PR. we can discuss the >>>> details in the PR itself before merging. In particular, I am wondering if >>>> the time travel clarification can be add to the existing `snapshots` >>>> section of the spec (instead of adding a new `implementation notes` >>>> section) >>>> >>>> On Thu, Jul 18, 2024 at 3:54 PM Ryan Blue <[email protected]> >>>> wrote: >>>> >>>>> +1 >>>>> >>>>> Thanks, Micah! >>>>> >>>>> On Tue, Jul 16, 2024 at 7:04 AM Jean-Baptiste Onofré <[email protected]> >>>>> wrote: >>>>> >>>>>> +1 (non binding) >>>>>> >>>>>> Thanks ! >>>>>> Regards >>>>>> JB >>>>>> >>>>>> On Mon, Jul 15, 2024 at 10:35 PM Micah Kornfield < >>>>>> [email protected]> wrote: >>>>>> > >>>>>> > I'd like to raise on modifying the table specification with >>>>>> clarifications on time travel and equality deletes [1][2]. The PRs have >>>>>> links to prior mailing list discussions where there was apparent >>>>>> consensus >>>>>> that these were the expectations for functionality. >>>>>> > >>>>>> > Possible votes: >>>>>> > [ ] +1 Merge the PRs >>>>>> > [ ] +0 >>>>>> > [ ] -1 Do not merge the PRs because ... >>>>>> > >>>>>> > The vote will remain open for at least 72 hours. >>>>>> > >>>>>> > Thanks, >>>>>> > Micah >>>>>> > >>>>>> > [1] https://github.com/apache/iceberg/pull/8982 >>>>>> > [2] https://github.com/apache/iceberg/pull/8981 >>>>>> >>>>> >>>>> >>>>> -- >>>>> Ryan Blue >>>>> Databricks >>>>> >>>>
