+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 >>>>>> >>>>> >
