srowen commented on a change in pull request #26826:
[SPARK-30195][SQL][CORE][ML] Change some function, import definitions to work
with stricter compiler in Scala 2.13
URL: https://github.com/apache/spark/pull/26826#discussion_r356019937
##########
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
##########
@@ -26,7 +26,7 @@ import org.apache.hadoop.mapred.JobConf
import org.apache.hadoop.mapreduce._
import org.apache.hadoop.mapreduce.lib.input.FileSplit
import org.apache.hadoop.mapreduce.task.TaskAttemptContextImpl
-import org.apache.orc._
+import org.apache.orc.{OrcUtils => _, _}
Review comment:
Ah right, the issue is there is also an org.apache.org.OrcUtils, in addition
to the one in this class's package. The code below wants to use the latter, the
Spark OrcUtils. In 2.12, the local package class wins, but in 2.13 it doesn't,
so the wrong one is imported.
This import syntax says "don't import OrcUtils from org.apache.orc, but
import everything else"
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
