OpenAPI: Add SetPartitionStatisticsUpdate/RemovePartitionStatisticsUpdate to TableUpdate

[feefs]

I think this should be a follow-up to #9690 (set-partition-statistics and remove-partition-statistics were added to BaseUpdate's discriminator: YAML sequence but not TableUpdate's anyOf: YAML sequence) and #9240 (original refactor from enum: to discriminator:).

Without this, the JSON Schema I'm generating for my validator is incorrectly rejecting CommitTableRequest payloads with TableUpdate schema objects such as set-partition-statistics.

---

Ran my changes through:

make install
make generate
make lint

and everything passes.

[ebyhr]

These two entries already exist in ViewUpdate. Why should we add them to TableUpdate?

[feefs]

Thanks for the catch, they shouldn't be copied over.

[ebyhr]

👍

[feefs]

Addressed comments.

[feefs]

@ebyhr friendly bump, I've rebased on top of the latest changes.

[ebyhr]

Looks good to me.

@Fokko @singhpk234 @huaxingao Could you review this PR when you have time?

[singhpk234]

LGTM, i think its a miss mostly since java client supports it in ser-de, vote should not be required.
Lets see what others have to say !

[feefs]

@ebyhr @singhpk234 @huaxingao @Fokko another friendly bump, are the current approvals sufficient for merging?

[github-actions]

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.

[github-actions]

This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

[huaxingao]

I think this can be merged without a vote.

[huaxingao]

also cc @nastra

[nastra]

I think this can be merged without a vote.

Yes I agree, I don't think we need a vote here because we're just adding something that was initially missed to be added