+1 (binding) Thanks for doing this, Steven. Yufei
On Thu, Jul 11, 2024 at 10:16 AM Amogh Jahagirdar <2am...@gmail.com> wrote: > + 1 (non-binding). > > Thanks, > > Amogh Jahagirdar > > On Thu, Jul 11, 2024 at 10:25 AM Péter Váry <peter.vary.apa...@gmail.com> > wrote: > >> +1 (non-binding) >> >> On Thu, Jul 11, 2024, 17:31 Jack Ye <yezhao...@gmail.com> wrote: >> >>> +1 (binding) >>> >>> On Thu, Jul 11, 2024 at 3:37 AM Piotr Findeisen < >>> piotr.findei...@gmail.com> wrote: >>> >>>> it looks it's part of the spec that's not connected to the other parts >>>> of the spec (like "dead code") >>>> >>>> +1 (non binding) >>>> >>>> >>>> On Thu, 11 Jul 2024 at 08:30, Eduard Tudenhöfner < >>>> etudenhoef...@apache.org> wrote: >>>> >>>>> +1 (non-binding) >>>>> >>>>> On Thu, Jul 11, 2024 at 8:29 AM Ajantha Bhat <ajanthab...@gmail.com> >>>>> wrote: >>>>> >>>>>> +1 (non-binding) >>>>>> >>>>>> - Ajantha >>>>>> >>>>>> On Thu, Jul 11, 2024 at 11:02 AM Jean-Baptiste Onofré < >>>>>> j...@nanthrax.net> wrote: >>>>>> >>>>>>> +1 (non binding) >>>>>>> >>>>>>> Regards >>>>>>> JB >>>>>>> >>>>>>> On Thu, Jul 11, 2024 at 12:50 AM Steven Wu <stevenz...@gmail.com> >>>>>>> wrote: >>>>>>> > >>>>>>> > Following the latest community guidelines, I would like to start a >>>>>>> voting thread on removing the JSON spec for content file and file scan >>>>>>> task. Here is the PR for the spec change [1] >>>>>>> > >>>>>>> > This was previously discussed in the dev mailing list [2]. While >>>>>>> it is good to add the JSON serializer in iceberg-core for ContentFile >>>>>>> and >>>>>>> FileScanTask, their JSON formats don't need to be added to the core >>>>>>> table >>>>>>> spec. >>>>>>> > >>>>>>> > Please vote in the next 72 hours. >>>>>>> > >>>>>>> > Thanks, >>>>>>> > Steven >>>>>>> > >>>>>>> > [1] https://github.com/apache/iceberg/pull/9771 >>>>>>> > [2] >>>>>>> https://lists.apache.org/thread/2ty27yx4q0zlqd5h71cyyhb5k47yf9bv >>>>>>> > >>>>>>> >>>>>>
