> > Does this mean that we can't upgrade v2 table to v3 table in a lazy > approach? That means it's not a mere table metadata upgrade, but we need to > upgrade all partition statistics?
Renjie, it is simply a clarification in the spec. We have treated NULL as 0 in V2 while it wasn't very obvious. No behavior change. пн, 5 трав. 2025 р. о 14:16 Anton Okolnychyi <aokolnyc...@gmail.com> пише: > My apologies for not following up earlier. This vote passed with 16 +1 (5 > binding) and no 0/-1 votes. I am going to merge the spec PR later today. > > - Anton > > ср, 5 лют. 2025 р. о 22:14 Renjie Liu <liurenjie2...@gmail.com> пише: > >> >Make delete counts required to avoid ambiguity w.r.t NULL vs unknown. >> >> Thanks Anton for driving this. Does this mean that we can't upgrade v2 >> table to v3 table in a lazy approach? That means it's not a mere table >> metadata upgrade, but we need to upgrade all partition statistics? >> >> On Thu, Feb 6, 2025 at 1:10 AM Denny Lee <denny.g....@gmail.com> wrote: >> >>> +1 (non-binding) - love this, Anton! >>> >>> On Wed, Feb 5, 2025 at 8:54 AM Prashant Singh <prashant010...@gmail.com> >>> wrote: >>> >>>> +1 (non-binding) >>>> Thank you Anton ! >>>> >>>> Best, >>>> Prashant Singh >>>> >>>> On Tue, Feb 4, 2025 at 10:42 PM Jean-Baptiste Onofré <j...@nanthrax.net> >>>> wrote: >>>> >>>>> +1 (non binding) >>>>> >>>>> Regards >>>>> JB >>>>> >>>>> 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 >>>>> >>>>
