+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 >> <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 >> >
