+1 from my side.
I don't think we should be blocked on Parquet's finalization

On Tue, Nov 26, 2024, 02:48 Xavier Bai <x...@apache.org> wrote:

> +1, not-binding
>
> Russell Spitzer <russell.spit...@gmail.com> 于2024年11月26日周二 01:21写道:
>
>> I'm +1,
>>
>> 1. I don't think we are going to change our decision on whether to
>> include variants based on the timing of Parquet ratification
>> 2. We aren't going to formally close V3 Spec yet, so if we do end up in a
>> situation where we want to close the spec and Parquet has not removed the
>> tag, we can remove the variant from the spec then. (I think that scenario
>> is unlikely)
>> 3. There is very little in our change set here that specifically
>> references the Parquet spec except for our reference link to it.
>>
>> I don't think there is anything that will happen in the spec that will
>> change what we would include in the Iceberg Spec (especially in this PR)
>>
>> On Fri, Nov 22, 2024 at 5:10 PM Micah Kornfield <emkornfi...@gmail.com>
>> wrote:
>>
>>> My (non-binding) vote is -1 until the variant spec is formally
>>> adopted in Parquet.
>>>
>>> On Fri, Nov 22, 2024 at 2:51 PM Aihua Xu <aihu...@gmail.com> wrote:
>>>
>>>> Hi everyone,
>>>>
>>>> I've updated the Iceberg spec to include the new Variant type as part
>>>> of #10831 <https://github.com/apache/iceberg/pull/10831>. The changes
>>>> are basically complete. This is a heads-up about the upcoming change.
>>>> Please review and +1 to acknowledge, so we will merge.
>>>>
>>>> Thanks,
>>>> Aihua
>>>>
>>>

Reply via email to