GitHub user dongjoon-hyun opened a pull request:

    https://github.com/apache/spark/pull/23064

    [MINOR][SQL] Fix typo in CTAS plan database string

    ## What changes were proposed in this pull request?
    
    Since Spark 1.6.0, there was a redundant '}' character in CTAS string 
plan's database argument string; `default}`. This PR aims to fix it.
    
    ```scala
    scala> sc.version
    res1: String = 1.6.0
    
    scala> sql("create table t as select 1").explain
    == Physical Plan ==
    ExecutedCommand CreateTableAsSelect [Database:default}, TableName: t, 
InsertIntoHiveTable]
    +- Project [1 AS _c0#3]
       +- OneRowRelation$
    ```
    
    ## How was this patch tested?
    
    Manual.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dongjoon-hyun/spark SPARK-FIX

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/23064.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #23064
    
----
commit ad971b668e05b93d54263ed87c9845d646085751
Author: Dongjoon Hyun <dongjoon@...>
Date:   2018-11-17T02:37:21Z

    [MINOR][SQL] Fix typo in CTAS plan argString
    
    ```
    scala> sql("create table t as select 1").explain
    == Physical Plan ==
    ExecutedCommand CreateTableAsSelect [Database:default}, TableName: t, 
InsertIntoHiveTable]
    +- Project [1 AS _c0#3]
       +- OneRowRelation$
    
    scala> sc.version
    res1: String = 1.6.0
    ```

----


---

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

Reply via email to