[ https://issues.apache.org/jira/browse/BEAM-4622?focusedWorklogId=123691&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-123691 ]
ASF GitHub Bot logged work on BEAM-4622: ---------------------------------------- Author: ASF GitHub Bot Created on: 16/Jul/18 16:50 Start Date: 16/Jul/18 16:50 Worklog Time Spent: 10m Work Description: akedin commented on issue #5912: [BEAM-4622] Makes required to call Beam SQL expressions validation URL: https://github.com/apache/beam/pull/5912#issuecomment-405312301 LGTM ---------------------------------------------------------------- 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: 123691) Time Spent: 1h (was: 50m) > 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: 1h > 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)