gengliangwang commented on code in PR #50407:
URL: https://github.com/apache/spark/pull/50407#discussion_r2014665221


##########
sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/DescribeTableSuite.scala:
##########
@@ -548,6 +547,7 @@ trait DescribeTableSuiteBase extends 
command.DescribeTableSuiteBase
                |""".stripMargin
           spark.sql(tableCreationStr)
           val viewType = if (isTemp) "TEMP VIEW" else "VIEW"
+          spark.sql(s"SET spark.sql.ansi.enabled = true")

Review Comment:
   since ANSI is enabled by default, shall we test setting it as off?
   also, let's test multiple sql conf changes instead of just one



-- 
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: reviews-unsubscr...@spark.apache.org

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

Reply via email to