+1 (non-binding) On Wed, May 7, 2025 at 8:37 AM Daniel Weeks <dwe...@apache.org> wrote:
> +1 (binding) > > On Wed, May 7, 2025 at 7:24 AM Russell Spitzer <russell.spit...@gmail.com> > wrote: > >> +1 (bind) >> >> On Wed, May 7, 2025 at 7:32 AM Eduard Tudenhöfner < >> etudenhoef...@apache.org> wrote: >> >>> +1 (binding) >>> >>> On Wed, May 7, 2025 at 4:14 AM Gang Wu <ust...@gmail.com> wrote: >>> >>>> The clarification is simple and clear from the writer's perspective. >>>> >>>> CMIW, the implication is that reader should drop bbox with any NaN >>>> value regardless of the coordinate axis (in case of a writer bug). >>>> >>>> On Wed, May 7, 2025 at 6:21 AM Huang-Hsiang Cheng >>>> <hua...@apple.com.invalid> wrote: >>>> >>>>> +1 (non-binding) >>>>> >>>>> On May 6, 2025, at 2:53 PM, Ryan Blue <rdb...@gmail.com> wrote: >>>>> >>>>> +1 (binding) >>>>> >>>>> Thanks to Jia and Szehon for the quick turn-around getting this done! >>>>> >>>>> On Tue, May 6, 2025 at 2:37 PM Jia Yu <ji...@apache.org> wrote: >>>>> >>>>>> +1 (non-binding) >>>>>> >>>>>> Thanks for putting this together! >>>>>> >>>>>> Jia Yu >>>>>> >>>>>> On Tue, May 6, 2025 at 2:09 PM Szehon Ho <szehon.apa...@gmail.com> >>>>>> wrote: >>>>>> >>>>>>> Hi everyone, >>>>>>> >>>>>>> As discussed briefly in >>>>>>> https://lists.apache.org/thread/ncj0xjh2ct5xvovn4tzc45lkm1wbmorq, >>>>>>> there is a minor clarification for geo type bounds that we want to get >>>>>>> in >>>>>>> for finalizing V3 spec. We want to clarify the behavior of null/NaN >>>>>>> coordinate values in geo objects. There can be many coordinate values >>>>>>> in a >>>>>>> geo object, only the null/NaN ones are skipped, the rest are not >>>>>>> skipped. >>>>>>> >>>>>>> I would like to raise a vote for this minor change to V3 spec. The >>>>>>> PR is: https://github.com/apache/iceberg/pull/12956. >>>>>>> >>>>>>> This is going on in parallel with a parquet-format pr: >>>>>>> https://github.com/apache/parquet-format/pull/494 >>>>>>> >>>>>>> This vote will be open for at least 72 hours. >>>>>>> >>>>>>> [ ] +1 Add these simplifications to the V3 Geo spec >>>>>>> [ ] +0 >>>>>>> [ ] -1 I have questions and/or concerns >>>>>>> >>>>>>> Thanks, >>>>>>> Szehon >>>>>>> >>>>>> >>>>>
