rdblue commented on PR #6267: URL: https://github.com/apache/iceberg/pull/6267#issuecomment-1347376734
> I think we still need an interface to get the current stats file for snapshot id currentStatisticsFiles() No, I don't think this is valuable. This depends too much on some definition of "current", which is about as useful as asking for _any_ stats file. This should be more specific. I agree that we want some way to get a stats file, but I don't think there's much value in the "current" idea. > It was just as per the previous existing interface in the builder. I wouldn't worry too much about intermediate representations. What we have in the current API is fine for now. I don't think that we should make the changes in this PR. -- 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
