rdblue commented on a change in pull request #1108:
URL: https://github.com/apache/iceberg/pull/1108#discussion_r441715741
##########
File path: build.gradle
##########
@@ -80,7 +80,7 @@ subprojects {
compileClasspath {
// do not exclude Guava so the bundle project can reference classes.
// the Spark module is also excluded because this breaks the Scala
compiler
- if (project.name != 'iceberg-bundled-guava' && project.name !=
'iceberg-spark') {
+ if (project.name != 'iceberg-bundled-guava' && project.name !=
'iceberg-spark' && project.name != 'iceberg-spark2') {
Review comment:
Not having Guava in the compile classpath breaks Scala compilation.
Right now, I'm planning to avoid adding Scala code to Spark 3, which is why it
is only for the current module and for the Spark 2 module where I plan to move
the current Scala code. See #1088.
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
