Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20133#discussion_r159168759
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala 
---
    @@ -1971,8 +1971,8 @@ abstract class DDLSuite extends QueryTest with 
SQLTestUtils {
               s"""
                  |CREATE TABLE t(a int, b int, c int, d int)
                  |USING parquet
    -             |PARTITIONED BY(a, b)
                  |LOCATION "${dir.toURI}"
    +             |PARTITIONED BY(a, b)
    --- End diff --
    
    Is it a relevant change? Since the PR is about ORDER-INSENSITIVENESS, can 
we keep the original code instead of making an irrelevant change like this?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to