[GitHub] [iceberg] rdblue commented on a change in pull request #2820: Spark: Moves Distribution and Ordering Utils to Spark3 Module

[GitBox <gi...@apache.org>]

rdblue commented on a change in pull request #2820:
URL: https://github.com/apache/iceberg/pull/2820#discussion_r694388118



##########
File path: build.gradle
##########
@@ -986,7 +986,14 @@ if (jdkVersion == '8') {
 }
 
 project(':iceberg-spark3') {
+  apply plugin: 'scala'
+
   sourceSets {
+    main {
+      // Use Scala Compiler for everything so we can do cross compilation within the module

Review comment:
       I don't think I understand why we can't use the default scala plugin settings. This is disabling Java compilation by removing srcDirs. Should we just remove the Java plugin?




-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org