[
https://issues.apache.org/jira/browse/BEAM-2282?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xu Mingmin updated BEAM-2282:
-----------------------------
Labels: dsl_sql_merge (was: )
> 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
> Labels: dsl_sql_merge
>
> 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)
