ajantha-bhat commented on PR #8502: URL: https://github.com/apache/iceberg/pull/8502#issuecomment-1861945339
@aokolnychyi: Thanks for the detailed review and merge. The next small PR to review will be https://github.com/apache/iceberg/pull/9284 (Spark 3.5: Ensure that partition stats files are considered for GC procedures) > I had only one comment about the access modifier for the base implementation class. I will check this in a followup. Again this follows the style from puffin files. I will also handle the applicable comments from this PR to stats files code (puffin files) in a follow up 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
