Re: [DISCUSS] Adding RemovePartitionSpecsUpdate update type to REST

[xianjin]

+1 from my side as well. Sent from my iPhone On Aug 20, 2024, at 9:09 AM, Yufei Gu <flyrain...@gmail.com> wrote:  +1, the new spec looks good to me. It seems like the client-side handling the heavy lifting of figuring out which spec to remove is a reasonable approach.  Yufei On Mon, Aug 19, 2024 at 4:01 PM Anton Okolnychyi <aokolnyc...@gmail.com> wrote: Seems reasonable to me. - Anton пн, 19 серп. 2024 р. о 15:19 Amogh Jahagirdar <2am...@gmail.com> пише: Hi all, There has been work [1] to enable users to remove historical partition specs which are not referenced in manifests as a form of metadata cleanup. As part of this, a new metadata update type RemovePartitionSpecsUpdate needs to be added to enable REST Catalogs to be able to perform this operation. I have a PR [2] with the REST spec change and am starting this thread to get feedback on the addition of this metadata update to the spec. Note: REST servers which receive an unknown type must fail with a 400 response as discussed/voted earlier. [3] [1] https://github.com/apache/iceberg/pull/10755  [2] https://github.com/apache/iceberg/pull/10846/ [3] https://lists.apache.org/thread/99lo7stnprchjzosjcq9k3mns1mq8fwc Thanks, Amogh Jahagirdar