The PR (https://github.com/apache/iceberg/pull/13513) has been merged. Thanks everyone.
Best, Kevin Liu On Wed, Jul 30, 2025 at 10:35 PM Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > +1 > > Regards > JB > > On Mon, Jul 28, 2025 at 12:38 PM Ajantha Bhat <ajanthab...@gmail.com> > wrote: > > > > Hi everyone, > > One of the users has raised a PR to update the table statistics (puffin > stats) spec. > > https://github.com/apache/iceberg/pull/13513 > > > > I have suggested a mailing list voting thread and also tagged the > original spec author. > > Since there was no response from them for a long time, I am taking it > forward. > > > > Spec mentions the snapshot id as String whereas java and python > implementations use Long. > > IMO, we can update the implementation to have a string to match the spec > and handle compatibility during read. > > But the spec is very old and definitely wrong (doesn't align with > regular snapshot id representation). > > Hence, I think updating the spec is the right option here as current > implementations like java and python library use long for snapshot id. > > > > Please take a look at the PR and cast your vote. > > > > - Ajantha > > >
