+1 On Tue, Feb 4, 2025 at 12:46 AM Honah J. <hon...@apache.org> wrote:
> +1 > > On Mon, Feb 3, 2025 at 11:42 PM Ajantha Bhat <ajanthab...@gmail.com> > wrote: > >> +1 >> >> On Tue, Feb 4, 2025 at 11:30 AM Eduard Tudenhöfner < >> etudenhoef...@apache.org> wrote: >> >>> +1 >>> >>> On Mon, Feb 3, 2025 at 8:33 PM Dongjoon Hyun <dongj...@apache.org> >>> wrote: >>> >>>> +1 for the proposal. >>>> >>>> Dongjoon >>>> >>>> > On Mon, Feb 3, 2025 at 2:35 PM ConradJam <czy...@apache.org> wrote: >>>> > >>>> > > +1 (non-binding) >>>> > > >>>> > > Steven Wu <stevenz...@gmail.com> 于2025年2月3日周一 14:08写道: >>>> > > >>>> > >> +1 >>>> > >> >>>> > >> The spec change makes sense. left a question in the PR. >>>> > >> >>>> > >> On Sun, Feb 2, 2025 at 8:52 PM roryqi <ror...@apache.org> wrote: >>>> > >> >>>> > >>> +1 >>>> > >>> >>>> > >>> Amogh Jahagirdar <2am...@gmail.com> 于2025年2月2日周日 10:16写道: >>>> > >>> >>>> > >>>> +1 >>>> > >>>> >>>> > >>>> On Sat, Feb 1, 2025 at 11:05 AM huaxin gao < >>>> huaxin.ga...@gmail.com> >>>> > >>>> wrote: >>>> > >>>> >>>> > >>>>> +1 (non-binding) >>>> > >>>>> >>>> > >>>>> On Sat, Feb 1, 2025 at 8:50 AM Manish Malhotra < >>>> > >>>>> manish.malhotra.w...@gmail.com> wrote: >>>> > >>>>> >>>> > >>>>>> +1(nonbinding) >>>> > >>>>>> >>>> > >>>>>> On Sat, Feb 1, 2025 at 2:49 AM Russell Spitzer < >>>> > >>>>>> russell.spit...@gmail.com> wrote: >>>> > >>>>>> >>>> > >>>>>>> +1 >>>> > >>>>>>> >>>> > >>>>>>> On Sat, Feb 1, 2025 at 3:01 AM Anton Okolnychyi < >>>> > >>>>>>> aokolnyc...@gmail.com> wrote: >>>> > >>>>>>> >>>> > >>>>>>>> Hi all, >>>> > >>>>>>>> >>>> > >>>>>>>> I propose the following updates to our partition stats spec >>>> in V3: >>>> > >>>>>>>> >>>> > >>>>>>>> - Modify `position_delete_record_count` to include a sum of >>>> > >>>>>>>> position deletes across position delete files and DVs >>>> > >>>>>>>> - Keep `position_delete_file_count` to represent the number >>>> of >>>> > >>>>>>>> position delete files (ignoring DVs) >>>> > >>>>>>>> - Add `dv_count` to represent the number of DVs >>>> > >>>>>>>> - Make delete counts required to avoid ambiguity w.r.t NULL >>>> vs >>>> > >>>>>>>> unknown. >>>> > >>>>>>>> >>>> > >>>>>>>> Here is the PR with the spec update: >>>> > >>>>>>>> https://github.com/apache/iceberg/pull/12098 >>>> > >>>>>>>> >>>> > >>>>>>>> - Anton >>>> > >>>>>>>> >>>> > >>>>>>> >>>> > >>>> >>>