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