+1 Thanks for removing the redundant! Yufei
On Tue, Jan 21, 2025 at 9:28 AM Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > +1 (non binding) > > Thanks Christian ! > > Regards > JB > > On Tue, Jan 21, 2025 at 8:25 AM Christian Thiel > <christian.t.b...@gmail.com> wrote: > > > > Hi everyone, > > > > based on good feedback on the [DISCUSS] thread [1] I would like to raise > > a vote to deprecate the `snapshot-id` field of the `SetStatisticsUpdate` > > in the IRC. It is redundant with the `snapshot-id` field of the > > `StatisticsFile` of each `SetStatisticsUpdate`. > > > > Please vote in the next 72 hours: > > > > [ ] +1, commit the proposed spec changes > > [ ] 0 > > [ ] -1, do not make these changes because... > > > > > > Best regards, > > > > Christian > > > > [1] Mailing List: > > https://lists.apache.org/thread/phxjz196zbzg0fjpfkmnj0fpkshgh9z0 > > [2] PR: https://github.com/apache/iceberg/pull/12010 > > > > >
