Re: [PR] Core: Make totalRecordCount optional in PartitionStats [iceberg]

2025-03-07 Thread via GitHub
jbonofre commented on PR #12226: URL: https://github.com/apache/iceberg/pull/12226#issuecomment-2703264435 @aokolnychyi @Fokko wdyt ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] Core: Make totalRecordCount optional in PartitionStats [iceberg]

2025-03-07 Thread via GitHub
ajantha-bhat commented on PR #12226: URL: https://github.com/apache/iceberg/pull/12226#issuecomment-2706657194 We have already finalized on this change by discussing on https://github.com/apache/iceberg/pull/12098#discussion_r1940551771 and https://lists.apache.org/thread/1dzcv7989q0hchjy

[PR] Core: Make totalRecordCount optional in PartitionStats [iceberg]

2025-02-11 Thread via GitHub
ajantha-bhat opened a new pull request, #12226: URL: https://github.com/apache/iceberg/pull/12226 Spec was already optional for totalRecordCount. https://iceberg.apache.org/spec/#partition-statistics-file During the implementation, we decided to make all the counters to be initiali