LuciferYang commented on a change in pull request #35506:
URL: https://github.com/apache/spark/pull/35506#discussion_r805569380
##########
File path:
sql/core/src/test/scala/org/apache/spark/sql/execution/QueryExecutionSuite.scala
##########
@@ -261,4 +261,22 @@ class QueryExecutionSuite extends SharedSparkSession {
val cmdResultExec = projectQe.executedPlan.asInstanceOf[CommandResultExec]
assert(cmdResultExec.commandPhysicalPlan.isInstanceOf[ShowTablesExec])
}
+
+ test("SPARK-38198: check specify maxFields when call toFile method") {
Review comment:
Before this pr, the test will failed as follows:
```
tableScan.exists(((x$8: String) => x$8.contains("more fields"))) was false
Specify maxFields = 3 doesn't take effect when explainMode is cost
ScalaTestFailureLocation: org.apache.spark.sql.execution.QueryExecutionSuite
at (QueryExecutionSuite.scala:162)
org.scalatest.exceptions.TestFailedException: tableScan.exists(((x$8:
String) => x$8.contains("more fields"))) was false Specify maxFields = 3
doesn't take effect when explainMode is cost
```
--
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]
