+1 On Tue, Feb 25, 2025 at 7:31 AM Steven Wu <stevenz...@gmail.com> wrote:
> +1 > > On Mon, Feb 24, 2025 at 10:13 PM Péter Váry <peter.vary.apa...@gmail.com> > wrote: > >> +1 >> >> On Tue, Feb 25, 2025, 04:16 Steve Zhang <hongyue_zh...@apple.com.invalid> >> wrote: >> >>> +1 (nb) >>> Thanks, >>> Steve Zhang >>> >>> >>> >>> On Feb 24, 2025, at 6:32 PM, Renjie Liu <liurenjie2...@gmail.com> wrote: >>> >>> +1 >>> >>> On Tue, Feb 25, 2025 at 7:00 AM Szehon Ho <szehon.apa...@gmail.com> >>> wrote: >>> >>>> +1 >>>> >>>> Thanks >>>> Szehon >>>> >>>> On Mon, Feb 24, 2025 at 2:52 PM rdb...@gmail.com <rdb...@gmail.com> >>>> wrote: >>>> >>>>> +1 >>>>> >>>>> On Mon, Feb 24, 2025 at 12:26 PM Daniel Weeks <dwe...@apache.org> >>>>> wrote: >>>>> >>>>>> +1 >>>>>> >>>>>> On Mon, Feb 24, 2025, 11:00 AM Russell Spitzer < >>>>>> russell.spit...@gmail.com> wrote: >>>>>> >>>>>>> +1 >>>>>>> >>>>>>> On Mon, Feb 24, 2025 at 12:55 PM Fokko Driesprong <fo...@apache.org> >>>>>>> wrote: >>>>>>> >>>>>>>> Hi everyone, >>>>>>>> >>>>>>>> Recently, there was confusion >>>>>>>> <https://lists.apache.org/thread/w403yzmpy9n9swcpwl2dcj2skdc0dskg> >>>>>>>> about >>>>>>>> valid values for the current-snapshot-id, which led to implementation >>>>>>>> notes <https://github.com/apache/iceberg/pull/12334> in the spec. >>>>>>>> Thanks for the reviews so far and to Daniel for the additional >>>>>>>> historical >>>>>>>> context on snapshot ID generation. Since there were a couple of >>>>>>>> reviews, and everything has been addressed, I would like to raise a >>>>>>>> vote to >>>>>>>> add this to the spec. >>>>>>>> >>>>>>>> This vote will be open for at least 72 hours. >>>>>>>> >>>>>>>> [ ] +1 Add the implementation notes to the spec >>>>>>>> [ ] +0 >>>>>>>> [ ] -1 I have questions and/or concerns >>>>>>>> >>>>>>>> Kind regards, >>>>>>>> Fokko >>>>>>>> >>>>>>> >>>
