Github user MaxGekk commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22233#discussion_r213419455
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala 
---
    @@ -52,23 +52,24 @@ class InMemoryCatalogedDDLSuite extends DDLSuite with 
SharedSQLContext with Befo
       protected override def generateTable(
           catalog: SessionCatalog,
           name: TableIdentifier,
    -      isDataSource: Boolean = true): CatalogTable = {
    +      isDataSource: Boolean = true,
    +      partitionCols: Seq[String] = Seq("a", "b")): CatalogTable = {
    --- End diff --
    
    @gatorsmile Yes, the changes are related to an existing test which was 
modified to reproduce the issue. In particular, this line is related to support 
of any number of partition columns. 


---

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

Reply via email to