You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Xu Mingmin (JIRA)" <ji...@apache.org> on 2017/05/12 16:27:04 UTC
[jira] [Created] (BEAM-2282) logic in function accept in
BeamSqlExpression
Xu Mingmin created BEAM-2282:
--------------------------------
Summary: 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
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.3.15#6346)