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

    https://github.com/apache/spark/pull/19392#discussion_r141949849
  
    --- Diff: 
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
    @@ -510,11 +510,15 @@ rowFormat
         ;
     
     tableIdentifier
    -    : (db=identifier '.')? table=identifier
    +    : (db=identifierPart '.')? table=identifierPart
    --- End diff --
    
    Hi, @cloud-fan .
    According to the PR title, the scope is **TABLE** name only, isn't it?
    Please remove the change on `db=identifierPart` and the test cases.
    
    [CREATE 
DATABASE](https://github.com/apache/spark/pull/19392/files#diff-8c1cb2af4aa1109e08481dae79052cc3R74)/DROP
 DATABASE/USE DATABASE still doesn't allow it.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to