LuciferYang commented on a change in pull request #30351:
URL: https://github.com/apache/spark/pull/30351#discussion_r527364012
##########
File path: pom.xml
##########
@@ -164,6 +164,7 @@
<commons.collections.version>3.2.2</commons.collections.version>
<scala.version>2.12.10</scala.version>
<scala.binary.version>2.12</scala.binary.version>
+ <scalac.arg.unused-imports>-Ywarn-unused-import</scalac.arg.unused-imports>
Review comment:
> Shall we just move it to SparkBuild.scala and let Maven doesn't care
about that?
It may be useful for Maven, it can also help Maven users to check this
although it is only warnings with maven build
----------------------------------------------------------------
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: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
