You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@iceberg.apache.org by Amogh Jahagirdar <2a...@gmail.com> on 2024/08/26 18:03:04 UTC
[VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type so
that removing partition specs update operation against REST catalogs can be
supported. See the discuss thread [2] for more context.
Please vote on merging this change. The vote will remain open for at least
72 hours.
[] +1
[] +0
[] -1, do not merge because ...
[1] https://github.com/apache/iceberg/pull/10846
[2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
Thanks,
Amogh Jahagirdar
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by Amogh Jahagirdar <2a...@gmail.com>.
Thanks all, with 5 binding and 3 non-binding votes the vote passed. The PR
has been merged.
Thanks,
Amogh Jahagirdar
On Thu, Aug 29, 2024 at 10:03 AM Daniel Weeks <dw...@apache.org> wrote:
> +1 (binding)
>
> On Wed, Aug 28, 2024 at 8:33 AM Jack Ye <ye...@gmail.com> wrote:
>
>> +1 (binding)
>>
>> On Tue, Aug 27, 2024 at 5:21 AM roryqi <ro...@apache.org> wrote:
>>
>>> +1
>>>
>>> Manu Zhang <ow...@gmail.com> 于2024年8月27日周二 11:44写道：
>>>
>>>> +1 (non-binding)
>>>>
>>>> On Tue, Aug 27, 2024 at 11:00 AM xianjin <xi...@apache.org> wrote:
>>>>
>>>>> +1 (non-binding)
>>>>> Sent from my iPhone
>>>>>
>>>>> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong <fo...@apache.org>
>>>>> wrote:
>>>>>
>>>>>
>>>>> +1
>>>>>
>>>>> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu <fl...@gmail.com>:
>>>>>
>>>>>> +1
>>>>>> Yufei
>>>>>>
>>>>>>
>>>>>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue
>>>>>> <bl...@databricks.com.invalid> wrote:
>>>>>>
>>>>>>> +1
>>>>>>>
>>>>>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update
>>>>>>>> type so that removing partition specs update operation against REST
>>>>>>>> catalogs can be supported. See the discuss thread [2] for more context.
>>>>>>>>
>>>>>>>> Please vote on merging this change. The vote will remain open for
>>>>>>>> at least 72 hours.
>>>>>>>>
>>>>>>>> [] +1
>>>>>>>> [] +0
>>>>>>>> [] -1, do not merge because ...
>>>>>>>>
>>>>>>>> [1] https://github.com/apache/iceberg/pull/10846
>>>>>>>> [2]
>>>>>>>> https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>>
>>>>>>>> Amogh Jahagirdar
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Ryan Blue
>>>>>>> Databricks
>>>>>>>
>>>>>>
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by Daniel Weeks <dw...@apache.org>.
+1 (binding)
On Wed, Aug 28, 2024 at 8:33 AM Jack Ye <ye...@gmail.com> wrote:
> +1 (binding)
>
> On Tue, Aug 27, 2024 at 5:21 AM roryqi <ro...@apache.org> wrote:
>
>> +1
>>
>> Manu Zhang <ow...@gmail.com> 于2024年8月27日周二 11:44写道：
>>
>>> +1 (non-binding)
>>>
>>> On Tue, Aug 27, 2024 at 11:00 AM xianjin <xi...@apache.org> wrote:
>>>
>>>> +1 (non-binding)
>>>> Sent from my iPhone
>>>>
>>>> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong <fo...@apache.org> wrote:
>>>>
>>>>
>>>> +1
>>>>
>>>> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu <fl...@gmail.com>:
>>>>
>>>>> +1
>>>>> Yufei
>>>>>
>>>>>
>>>>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue <bl...@databricks.com.invalid>
>>>>> wrote:
>>>>>
>>>>>> +1
>>>>>>
>>>>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com>
>>>>>> wrote:
>>>>>>
>>>>>>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type
>>>>>>> so that removing partition specs update operation against REST catalogs can
>>>>>>> be supported. See the discuss thread [2] for more context.
>>>>>>>
>>>>>>> Please vote on merging this change. The vote will remain open for
>>>>>>> at least 72 hours.
>>>>>>>
>>>>>>> [] +1
>>>>>>> [] +0
>>>>>>> [] -1, do not merge because ...
>>>>>>>
>>>>>>> [1] https://github.com/apache/iceberg/pull/10846
>>>>>>> [2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>>>>>>>
>>>>>>>
>>>>>>> Thanks,
>>>>>>>
>>>>>>> Amogh Jahagirdar
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Ryan Blue
>>>>>> Databricks
>>>>>>
>>>>>
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by Jack Ye <ye...@gmail.com>.
+1 (binding)
On Tue, Aug 27, 2024 at 5:21 AM roryqi <ro...@apache.org> wrote:
> +1
>
> Manu Zhang <ow...@gmail.com> 于2024年8月27日周二 11:44写道：
>
>> +1 (non-binding)
>>
>> On Tue, Aug 27, 2024 at 11:00 AM xianjin <xi...@apache.org> wrote:
>>
>>> +1 (non-binding)
>>> Sent from my iPhone
>>>
>>> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong <fo...@apache.org> wrote:
>>>
>>>
>>> +1
>>>
>>> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu <fl...@gmail.com>:
>>>
>>>> +1
>>>> Yufei
>>>>
>>>>
>>>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue <bl...@databricks.com.invalid>
>>>> wrote:
>>>>
>>>>> +1
>>>>>
>>>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com>
>>>>> wrote:
>>>>>
>>>>>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type
>>>>>> so that removing partition specs update operation against REST catalogs can
>>>>>> be supported. See the discuss thread [2] for more context.
>>>>>>
>>>>>> Please vote on merging this change. The vote will remain open for
>>>>>> at least 72 hours.
>>>>>>
>>>>>> [] +1
>>>>>> [] +0
>>>>>> [] -1, do not merge because ...
>>>>>>
>>>>>> [1] https://github.com/apache/iceberg/pull/10846
>>>>>> [2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> Amogh Jahagirdar
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Ryan Blue
>>>>> Databricks
>>>>>
>>>>
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by roryqi <ro...@apache.org>.
+1
Manu Zhang <ow...@gmail.com> 于2024年8月27日周二 11:44写道：
> +1 (non-binding)
>
> On Tue, Aug 27, 2024 at 11:00 AM xianjin <xi...@apache.org> wrote:
>
>> +1 (non-binding)
>> Sent from my iPhone
>>
>> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong <fo...@apache.org> wrote:
>>
>>
>> +1
>>
>> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu <fl...@gmail.com>:
>>
>>> +1
>>> Yufei
>>>
>>>
>>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue <bl...@databricks.com.invalid>
>>> wrote:
>>>
>>>> +1
>>>>
>>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com>
>>>> wrote:
>>>>
>>>>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type
>>>>> so that removing partition specs update operation against REST catalogs can
>>>>> be supported. See the discuss thread [2] for more context.
>>>>>
>>>>> Please vote on merging this change. The vote will remain open for at
>>>>> least 72 hours.
>>>>>
>>>>> [] +1
>>>>> [] +0
>>>>> [] -1, do not merge because ...
>>>>>
>>>>> [1] https://github.com/apache/iceberg/pull/10846
>>>>> [2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>>>>>
>>>>> Thanks,
>>>>>
>>>>> Amogh Jahagirdar
>>>>>
>>>>
>>>>
>>>> --
>>>> Ryan Blue
>>>> Databricks
>>>>
>>>
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by Manu Zhang <ow...@gmail.com>.
+1 (non-binding)
On Tue, Aug 27, 2024 at 11:00 AM xianjin <xi...@apache.org> wrote:
> +1 (non-binding)
> Sent from my iPhone
>
> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong <fo...@apache.org> wrote:
>
>
> +1
>
> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu <fl...@gmail.com>:
>
>> +1
>> Yufei
>>
>>
>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue <bl...@databricks.com.invalid>
>> wrote:
>>
>>> +1
>>>
>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com>
>>> wrote:
>>>
>>>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type so
>>>> that removing partition specs update operation against REST catalogs can be
>>>> supported. See the discuss thread [2] for more context.
>>>>
>>>> Please vote on merging this change. The vote will remain open for at
>>>> least 72 hours.
>>>>
>>>> [] +1
>>>> [] +0
>>>> [] -1, do not merge because ...
>>>>
>>>> [1] https://github.com/apache/iceberg/pull/10846
>>>> [2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>>>>
>>>> Thanks,
>>>>
>>>> Amogh Jahagirdar
>>>>
>>>
>>>
>>> --
>>> Ryan Blue
>>> Databricks
>>>
>>
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by xianjin <xi...@apache.org>.
+1 (non-binding)
Sent from my iPhone
> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong <fo...@apache.org> wrote:
>
>
>
>
> +1
>
>
>
>
>
> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu
> <[flyrain000@gmail.com](mailto:flyrain000@gmail.com)>:
>
>
>> +1
>
>>
>> Yufei
>>
>>
>
>>
>>
>
>>
>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue <bl...@databricks.com.invalid>
wrote:
>
>>
>>> +1
>
>>>
>>>
>
>>>
>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar
<[2amogh@gmail.com](mailto:2amogh@gmail.com)> wrote:
>
>>>
>>>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type so
that removing partition specs update operation against REST catalogs can be
supported. See the discuss thread [2] for more context.
>
>>>>
>>>>
>
>>>>
>>>> Please vote on merging this change. The vote will remain open for at
least 72 hours.
>>>>
>>>>
>
>>>>
>>>> [] +1
>>>>
>>>> [] +0
>>>>
>>>> [] -1, do not merge because ...
>>>>
>>>>
>
>>>>
>>>> [1] <https://github.com/apache/iceberg/pull/10846>
>
>>>>
>>>> [2] <https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln>
>>>>
>>>>
>
>>>>
>>>> Thanks,
>>>>
>>>>
>
>>>>
>>>> Amogh Jahagirdar
>>>
>>>
>
>>>
>>>
>
>>>
>>> \--
>
>>>
>>> Ryan Blue
>>>
>>> Databricks
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by Fokko Driesprong <fo...@apache.org>.
+1
Op ma 26 aug 2024 om 22:00 schreef Yufei Gu <fl...@gmail.com>:
> +1
> Yufei
>
>
> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue <bl...@databricks.com.invalid>
> wrote:
>
>> +1
>>
>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com>
>> wrote:
>>
>>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type so
>>> that removing partition specs update operation against REST catalogs can be
>>> supported. See the discuss thread [2] for more context.
>>>
>>> Please vote on merging this change. The vote will remain open for at
>>> least 72 hours.
>>>
>>> [] +1
>>> [] +0
>>> [] -1, do not merge because ...
>>>
>>> [1] https://github.com/apache/iceberg/pull/10846
>>> [2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>>>
>>> Thanks,
>>>
>>> Amogh Jahagirdar
>>>
>>
>>
>> --
>> Ryan Blue
>> Databricks
>>
>
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by Yufei Gu <fl...@gmail.com>.
+1
Yufei
On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue <bl...@databricks.com.invalid>
wrote:
> +1
>
> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com>
> wrote:
>
>> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type so
>> that removing partition specs update operation against REST catalogs can be
>> supported. See the discuss thread [2] for more context.
>>
>> Please vote on merging this change. The vote will remain open for at
>> least 72 hours.
>>
>> [] +1
>> [] +0
>> [] -1, do not merge because ...
>>
>> [1] https://github.com/apache/iceberg/pull/10846
>> [2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>>
>> Thanks,
>>
>> Amogh Jahagirdar
>>
>
>
> --
> Ryan Blue
> Databricks
>
Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type
Posted by Ryan Blue <bl...@databricks.com.INVALID>.
+1
On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <2a...@gmail.com> wrote:
> I've opened a PR [1] to add a RemovePartitionSpecsUpdate update type so
> that removing partition specs update operation against REST catalogs can be
> supported. See the discuss thread [2] for more context.
>
> Please vote on merging this change. The vote will remain open for at
> least 72 hours.
>
> [] +1
> [] +0
> [] -1, do not merge because ...
>
> [1] https://github.com/apache/iceberg/pull/10846
> [2] https://lists.apache.org/thread/2lo59wkhn8cm29wmokfmg2f18934qnln
>
> Thanks,
>
> Amogh Jahagirdar
>
--
Ryan Blue
Databricks