Copilot commented on code in PR #11565:
URL:
https://github.com/apache/incubator-gluten/pull/11565#discussion_r2766732585
##########
gluten-ut/spark41/src/test/scala/org/apache/spark/sql/GlutenLogQuerySuite.scala:
##########
@@ -16,4 +16,42 @@
*/
package org.apache.spark.sql
-class GlutenLogQuerySuite extends LogQuerySuite with GlutenSQLTestsTrait {}
+import org.apache.spark.internal.LogKeys
Review Comment:
The `MDC` function is used on line 33 but it is not imported. In Spark 4.0
(line 19 of
gluten-ut/spark40/src/test/scala/org/apache/spark/sql/GlutenLogQuerySuite.scala),
`MDC` is explicitly imported from `org.apache.spark.internal`, but this import
is missing in the Spark 4.1 version. This will likely cause a compilation
error. Add the import statement: `import org.apache.spark.internal.MDC`
```suggestion
import org.apache.spark.internal.LogKeys
import org.apache.spark.internal.MDC
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
