Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type

2024-08-30 Thread Amogh Jahagirdar 
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  wrote:

> +1 (binding)
>
> On Wed, Aug 28, 2024 at 8:33 AM Jack Ye  wrote:
>
>> +1 (binding)
>>
>> On Tue, Aug 27, 2024 at 5:21 AM roryqi  wrote:
>>
>>> +1
>>>
>>> Manu Zhang  于2024年8月27日周二 11:44写道：
>>>
 +1 (non-binding)

 On Tue, Aug 27, 2024 at 11:00 AM xianjin  wrote:

> +1 (non-binding)
> Sent from my iPhone
>
> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong 
> wrote:
>
> 
> +1
>
> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu :
>
>> +1
>> Yufei
>>
>>
>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue
>>  wrote:
>>
>>> +1
>>>
>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]>
>>> 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

2024-08-29 Thread Daniel Weeks 
+1 (binding)

On Wed, Aug 28, 2024 at 8:33 AM Jack Ye  wrote:

> +1 (binding)
>
> On Tue, Aug 27, 2024 at 5:21 AM roryqi  wrote:
>
>> +1
>>
>> Manu Zhang  于2024年8月27日周二 11:44写道：
>>
>>> +1 (non-binding)
>>>
>>> On Tue, Aug 27, 2024 at 11:00 AM xianjin  wrote:
>>>
 +1 (non-binding)
 Sent from my iPhone

 On Aug 27, 2024, at 4:22 AM, Fokko Driesprong  wrote:

 
 +1

 Op ma 26 aug 2024 om 22:00 schreef Yufei Gu :

> +1
> Yufei
>
>
> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue 
> wrote:
>
>> +1
>>
>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]>
>> 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

2024-08-28 Thread Jack Ye 
+1 (binding)

On Tue, Aug 27, 2024 at 5:21 AM roryqi  wrote:

> +1
>
> Manu Zhang  于2024年8月27日周二 11:44写道：
>
>> +1 (non-binding)
>>
>> On Tue, Aug 27, 2024 at 11:00 AM xianjin  wrote:
>>
>>> +1 (non-binding)
>>> Sent from my iPhone
>>>
>>> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong  wrote:
>>>
>>> 
>>> +1
>>>
>>> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu :
>>>
 +1
 Yufei


 On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue 
 wrote:

> +1
>
> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]>
> 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

2024-08-27 Thread roryqi 
+1

Manu Zhang  于2024年8月27日周二 11:44写道：

> +1 (non-binding)
>
> On Tue, Aug 27, 2024 at 11:00 AM xianjin  wrote:
>
>> +1 (non-binding)
>> Sent from my iPhone
>>
>> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong  wrote:
>>
>> 
>> +1
>>
>> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu :
>>
>>> +1
>>> Yufei
>>>
>>>
>>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue 
>>> wrote:
>>>
 +1

 On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]>
 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

2024-08-26 Thread Manu Zhang 
+1 (non-binding)

On Tue, Aug 27, 2024 at 11:00 AM xianjin  wrote:

> +1 (non-binding)
> Sent from my iPhone
>
> On Aug 27, 2024, at 4:22 AM, Fokko Driesprong  wrote:
>
> 
> +1
>
> Op ma 26 aug 2024 om 22:00 schreef Yufei Gu :
>
>> +1
>> Yufei
>>
>>
>> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue 
>> wrote:
>>
>>> +1
>>>
>>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]>
>>> 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

2024-08-26 Thread xianjin 
+1 (non-binding)Sent from my iPhoneOn Aug 27, 2024, at 4:22 AM, Fokko Driesprong  wrote:+1Op ma 26 aug 2024 om 22:00 schreef Yufei Gu :+1YufeiOn Mon, Aug 26, 2024 at 11:06 AM Ryan Blue  wrote:+1On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]> 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 BlueDatabricks

Re: [VOTE] Merge REST Spec change to add RemovePartitionSpecsUpdate update type

2024-08-26 Thread Fokko Driesprong 
+1

Op ma 26 aug 2024 om 22:00 schreef Yufei Gu :

> +1
> Yufei
>
>
> On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue 
> wrote:
>
>> +1
>>
>> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]>
>> 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

2024-08-26 Thread Yufei Gu 
+1
Yufei


On Mon, Aug 26, 2024 at 11:06 AM Ryan Blue 
wrote:

> +1
>
> On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]>
> 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

2024-08-26 Thread Ryan Blue 
+1

On Mon, Aug 26, 2024 at 11:04 AM Amogh Jahagirdar <[email protected]> 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