rdblue commented on issue #1567: URL: https://github.com/apache/iceberg/issues/1567#issuecomment-706608984
@simonsssu, if we want additional Guava classes, then we add them to [`GuavaClasses`](https://github.com/apache/iceberg/blob/425b10cea34eef11cca9cf0d237e02274f6dc958/bundled-guava/src/main/java/org/apache/iceberg/GuavaClasses.java) so that they aren't removed when the Jar is minimized. Also, we are trying to avoid depending more on Guava because it causes so many runtime issues. I think that the current build is a reasonable way to go, but maybe I've misunderstood your concern? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
