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