ajantha-bhat commented on PR #6267: URL: https://github.com/apache/iceberg/pull/6267#issuecomment-1343786472
> @ajantha-bhat, I'm fine with the update to the spec wording, but I don't think the rest of these changes are needed. I think we still need an interface to get the current stats file for snapshot id `currentStatisticsFiles()` > There should be only one stats file per snapshot in metadata. There is no value in having multiple stats files and implementations should merge them. Maybe true. But It was just as per the previous existing interface in the builder. we had `Map<Long, List<StatisticsFile>> statisticsFiles` in builder. If we wanted one stats file per interface then no need to have list for map value? Also is it possible that there will be one partition level stats (avro/parquet file) and there is one table level NDV file per snapshot. In this case, no need to merge them but can be used based on the need. So, still ok to have multiple stats per snapshot id? -- 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 comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
