+1 On Tue, Aug 20, 2024 at 1:56 AM Fokko Driesprong <fo...@apache.org> wrote:
> +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 >>>> >>> -- Ryan Blue Databricks
