Repository navigation
Fix TableUpdate union to include partition statistics updates - #15115
amogh-jahagirdar merged 1 commit into
Conversation
…csUpdate in TableUpdate union The `SetPartitionStatisticsUpdate` and `RemovePartitionStatisticsUpdate` were defined in the OpenAPI spec but missing from the `TableUpdate` union. This caused the generated Python client and other consumers to not recognize these updates as valid table updates. This commit adds them to the `TableUpdate` union in `rest-catalog-open-api.yaml` and regenerates `rest-catalog-open-api.py`.
amogh-jahagirdar
left a comment
amogh-jahagirdar
left a comment
There was a problem hiding this comment.
Thanks @logesh45 , I agree with adding this, though I think we may just want to send a note on the dev list so that the community is aware.
Hi, Sorry I'm new here. how do I do that? what is the criteria to get this merged? |
|
@logesh45 no problem, I went ahead and just kicked a thread off https://lists.apache.org/thread/dly4g6wp5c83czvxyl6fhhncb7kc85dk since this is a simple enough correction, where it was an oversight on the original PR. We probably should just be able to merge, but I wanted to make sure folks were aware before just merging this. |
|
Thanks @logesh45 and everyone for reviewing, I'll go ahead and merge since there's been no objections on the mailing list thread |
…atisticsUpdate in TableUpdate union (apache#15115) The `SetPartitionStatisticsUpdate` and `RemovePartitionStatisticsUpdate` were defined in the OpenAPI spec but missing from the `TableUpdate` union. This caused the generated Python client and other consumers to not recognize these updates as valid table updates. This commit adds them to the `TableUpdate` union in `rest-catalog-open-api.yaml` and regenerates `rest-catalog-open-api.py`.
11 participants
fix: Include SetPartitionStatisticsUpdate and RemovePartitionStatisticsUpdate in TableUpdate union
The
SetPartitionStatisticsUpdateandRemovePartitionStatisticsUpdatewere defined in the OpenAPI spec but missing from theTableUpdateunion. This caused the generated Python client and other consumers to not recognize these updates as valid table updates. This commit adds them to theTableUpdateunion inrest-catalog-open-api.yamland regeneratesrest-catalog-open-api.py.logesh45#1