+1 (binding) Thanks Szehon
On Wed, May 7, 2025 at 10:41 AM huaxin gao <huaxin.ga...@gmail.com> wrote: > +1 (non-binding) > > On Wed, May 7, 2025 at 9:29 AM Denny Lee <denny.g....@gmail.com> wrote: > >> +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 >>>>>>>>> >>>>>>>> >>>>>>>
