You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@iceberg.apache.org by Micah Kornfield <em...@gmail.com> on 2024/08/02 18:06:20 UTC
[VOTE] Merge specification clarifications on reading/writing partition values
I've opened a PR [1] to clarify that partition columns must always be
written by implementations and that for identity transformed partition
values, the metadata from the manifest file must be used.
Please vote on merging this change. The vote will remain open for at least
72 hours.
[] +1
[] +0
[] -1, do not merge because ...
My vote is +1
Thanks,
Micah
[1] https://github.com/apache/iceberg/pull/10835
Re: [VOTE] Merge specification clarifications on reading/writing partition values
Posted by Daniel Weeks <dw...@apache.org>.
+1 (binding)
On Fri, Aug 2, 2024 at 1:25 PM Ryan Blue <bl...@databricks.com.invalid>
wrote:
> +1 (binding)
>
> On Fri, Aug 2, 2024 at 12:03 PM Yufei Gu <fl...@gmail.com> wrote:
>
>> +1 (binding)
>> Yufei
>>
>>
>> On Fri, Aug 2, 2024 at 11:18 AM Prashant Singh <pr...@gmail.com>
>> wrote:
>>
>>> +1 (non-binding)
>>> Thanks Micah !
>>>
>>> Regards,
>>> Prashant
>>>
>>> On Fri, Aug 2, 2024 at 11:06 AM Micah Kornfield <em...@gmail.com>
>>> wrote:
>>>
>>>> I've opened a PR [1] to clarify that partition columns must always be
>>>> written by implementations and that for identity transformed partition
>>>> values, the metadata from the manifest file must be used.
>>>>
>>>> Please vote on merging this change. The vote will remain open for at
>>>> least 72 hours.
>>>>
>>>> [] +1
>>>> [] +0
>>>> [] -1, do not merge because ...
>>>>
>>>> My vote is +1
>>>>
>>>> Thanks,
>>>> Micah
>>>>
>>>>
>>>> [1] https://github.com/apache/iceberg/pull/10835
>>>>
>>>
>
> --
> Ryan Blue
> Databricks
>
Re: [VOTE] Merge specification clarifications on reading/writing partition values
Posted by Ryan Blue <bl...@databricks.com.INVALID>.
+1 (binding)
On Fri, Aug 2, 2024 at 12:03 PM Yufei Gu <fl...@gmail.com> wrote:
> +1 (binding)
> Yufei
>
>
> On Fri, Aug 2, 2024 at 11:18 AM Prashant Singh <pr...@gmail.com>
> wrote:
>
>> +1 (non-binding)
>> Thanks Micah !
>>
>> Regards,
>> Prashant
>>
>> On Fri, Aug 2, 2024 at 11:06 AM Micah Kornfield <em...@gmail.com>
>> wrote:
>>
>>> I've opened a PR [1] to clarify that partition columns must always be
>>> written by implementations and that for identity transformed partition
>>> values, the metadata from the manifest file must be used.
>>>
>>> Please vote on merging this change. The vote will remain open for at
>>> least 72 hours.
>>>
>>> [] +1
>>> [] +0
>>> [] -1, do not merge because ...
>>>
>>> My vote is +1
>>>
>>> Thanks,
>>> Micah
>>>
>>>
>>> [1] https://github.com/apache/iceberg/pull/10835
>>>
>>
--
Ryan Blue
Databricks
Re: [VOTE] Merge specification clarifications on reading/writing partition values
Posted by Yufei Gu <fl...@gmail.com>.
+1 (binding)
Yufei
On Fri, Aug 2, 2024 at 11:18 AM Prashant Singh <pr...@gmail.com>
wrote:
> +1 (non-binding)
> Thanks Micah !
>
> Regards,
> Prashant
>
> On Fri, Aug 2, 2024 at 11:06 AM Micah Kornfield <em...@gmail.com>
> wrote:
>
>> I've opened a PR [1] to clarify that partition columns must always be
>> written by implementations and that for identity transformed partition
>> values, the metadata from the manifest file must be used.
>>
>> Please vote on merging this change. The vote will remain open for at
>> least 72 hours.
>>
>> [] +1
>> [] +0
>> [] -1, do not merge because ...
>>
>> My vote is +1
>>
>> Thanks,
>> Micah
>>
>>
>> [1] https://github.com/apache/iceberg/pull/10835
>>
>
Re: [VOTE] Merge specification clarifications on reading/writing partition values
Posted by Prashant Singh <pr...@gmail.com>.
+1 (non-binding)
Thanks Micah !
Regards,
Prashant
On Fri, Aug 2, 2024 at 11:06 AM Micah Kornfield <em...@gmail.com>
wrote:
> I've opened a PR [1] to clarify that partition columns must always be
> written by implementations and that for identity transformed partition
> values, the metadata from the manifest file must be used.
>
> Please vote on merging this change. The vote will remain open for at
> least 72 hours.
>
> [] +1
> [] +0
> [] -1, do not merge because ...
>
> My vote is +1
>
> Thanks,
> Micah
>
>
> [1] https://github.com/apache/iceberg/pull/10835
>