feefs opened a new pull request, #14957: URL: https://github.com/apache/iceberg/pull/14957
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](https://github.com/apache/iceberg/blob/00046005889c66ffc860bae012d7fc560e8f040a/open-api/rest-catalog-open-api.yaml#L3571-L3574C22) such as `set-partition-statistics`. --- Ran my changes through: ``` make install make generate make lint ``` and everything passes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
