[ 
https://issues.apache.org/jira/browse/BEAM-2282?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Xu Mingmin closed BEAM-2282.
----------------------------
       Resolution: Fixed
         Assignee: James Xu
    Fix Version/s: Not applicable

> logic in function accept in BeamSqlExpression
> ---------------------------------------------
>
>                 Key: BEAM-2282
>                 URL: https://issues.apache.org/jira/browse/BEAM-2282
>             Project: Beam
>          Issue Type: Improvement
>          Components: dsl-sql
>            Reporter: Xu Mingmin
>            Assignee: James Xu
>              Labels: dsl_sql_merge
>             Fix For: Not applicable
>
>
> Method {{accept}} is used to make sure the SQL operator is supported in Beam 
> SQL. IMO it should only check the case that some scenarios are not supported 
> in BeamSQL even it's valid in Calcite SQL grammar.
> It's not necessary to my understand to validate SQL grammar as that's done 
> already in the parsing phase.  



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to