+1 Thanks for working on this Op di 20 aug 2024 om 04:16 schreef xianjin <xian...@apache.org>:
> +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 >>> <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 >>> >>