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

Reply via email to