jackye1995 commented on pull request #3463: URL: https://github.com/apache/iceberg/pull/3463#issuecomment-961277912
> changes make sense, what about preventing usage of org.apache.parquet.Preconditions? I believe this is also being used unintentionally at a few places sounds good, I also want to block the use of `org.apache.commons.io.IOUtils` in favor of Guava `ByteStreams`, let me put up another PR for these 2 to avoid too many changes in a single PR. I will go ahead to merge this one, thanks for the reviews! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
