[
https://issues.apache.org/jira/browse/BEAM-4622?focusedWorklogId=123562&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-123562
]
ASF GitHub Bot logged work on BEAM-4622:
----------------------------------------
Author: ASF GitHub Bot
Created on: 16/Jul/18 09:58
Start Date: 16/Jul/18 09:58
Worklog Time Spent: 10m
Work Description: aromanenko-dev commented on issue #5912: [BEAM-4622]
Makes required to call Beam SQL expressions validation
URL: https://github.com/apache/beam/pull/5912#issuecomment-405199636
@akedin Could you take a look on this?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 123562)
Time Spent: 50m (was: 40m)
> Many Beam SQL expressions never have their validation called
> ------------------------------------------------------------
>
> Key: BEAM-4622
> URL: https://issues.apache.org/jira/browse/BEAM-4622
> Project: Beam
> Issue Type: Bug
> Components: dsl-sql
> Reporter: Kenneth Knowles
> Assignee: Alexey Romanenko
> Priority: Major
> Labels: easyfix, newbie, starter
> Time Spent: 50m
> Remaining Estimate: 0h
>
> In {{BeamSqlFnExecutor}} there is a pattern where first the returned
> expression is assigned to a variable {{ret}} and then after a giant switch
> statement the validation is invoked. But there are many code paths that just
> call {{return}} and skip validation. This should be refactored so it is
> impossible to short-circuit on accident like this.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
