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

Reply via email to