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

    https://github.com/apache/spark/pull/19559#discussion_r146309332
  
    --- Diff: 
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
    @@ -585,6 +584,7 @@ primaryExpression
         | identifier                                                           
                    #columnReference
         | base=primaryExpression '.' fieldName=identifier                      
                    #dereference
         | '(' expression ')'                                                   
                    #parenthesizedExpression
    +    | name=(CURRENT_DATE | CURRENT_TIMESTAMP)                              
                    #timeFunctionCall
    --- End diff --
    
    Won't this break every use of `CURRENT_DATE`/`CURRENT_TIMESTAMP`? They will 
now be interpreted as an identifier.


---

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

Reply via email to