cloud-fan commented on a change in pull request #31434:
URL: https://github.com/apache/spark/pull/31434#discussion_r568413503
##########
File path:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/ShowPartitionsSuiteBase.scala
##########
@@ -161,7 +161,7 @@ trait ShowPartitionsSuiteBase extends QueryTest with
DDLCommandTestUtils {
sql(s"CREATE TABLE $t (col1 INT, p1 STRING) $defaultUsing PARTITIONED
BY (p1)")
sql(s"INSERT INTO TABLE $t PARTITION (p1 = null) SELECT 0")
runShowPartitionsSql(s"SHOW PARTITIONS $t", Row("p1=null") :: Nil)
- runShowPartitionsSql(s"SHOW PARTITIONS $t PARTITION (p1 = null)",
Row("p1=null") :: Nil)
+ checkAnswer(spark.sql(s"SELECT * FROM $t"), Row(0, "null"))
Review comment:
I don't know why we put the test in `ShowPartitionsSuite`, as the
correctness bug is more obvious in the table scan case. Additionally, this test
suite is master only and may bring troubles during backporting.
Shall we simply put the test in `SQLQuerySuite`?
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
