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)

Reply via email to