jacobmarble commented on PR #8971: URL: https://github.com/apache/iceberg/pull/8971#issuecomment-1802450224
> @jacobmarble can you break this into smaller commits? There are a ton of files changed here and I'm concerned about catching problems with such a large PR. > I agree with @rdblue > > For partition stat we had a PR dedicated for spec. As it's not necessary the same reviewers, I think it's good to have one PR for spec (even if it's very small) and PRs for api and impl. Happy to split this up. I tried to scope this to API and Core, but the cascade of failing tests led to this. -- 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
