You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@iceberg.apache.org by Amogh Jahagirdar <2a...@gmail.com> on 2026/01/30 19:24:51 UTC
[DISCUSS] Merging a correction to REST Spec to include Partition Stats Updates in TableUpdate
Hey folks, just wanted to start a thread on this change
https://github.com/apache/iceberg/pull/15115 . The spec already defined
partition stats commit update types, but forgot to include the references
to these types in the TableUpdate, which leads to some unexpected problems
for generated clients etc.
This just looks to have been an oversight on the original PR, and should
probably just be treated as a spec correction which should not require a
vote.
I just wanted to call it out on the dev list in case there are any
objections, otherwise I'll just go ahead and merge the PR in a few days.
Thanks,
Amogh Jahagirdar
Re: [DISCUSS] Merging a correction to REST Spec to include Partition Stats Updates in TableUpdate
Posted by Kevin Liu <ke...@apache.org>.
`SetPartitionStatisticsUpdate` and `RemovePartitionStatisticsUpdate` are
definitely both table updates. I think it's fair to treat it as a spec
correction.
Thanks for bringing it up to the devlist.
On Fri, Jan 30, 2026 at 11:25 AM Amogh Jahagirdar <2a...@gmail.com> wrote:
> Hey folks, just wanted to start a thread on this change
> https://github.com/apache/iceberg/pull/15115 . The spec already defined
> partition stats commit update types, but forgot to include the references
> to these types in the TableUpdate, which leads to some unexpected problems
> for generated clients etc.
>
> This just looks to have been an oversight on the original PR, and should
> probably just be treated as a spec correction which should not require a
> vote.
>
> I just wanted to call it out on the dev list in case there are any
> objections, otherwise I'll just go ahead and merge the PR in a few days.
>
> Thanks,
> Amogh Jahagirdar
>