Description
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.
Attachments
Issue Links
- links to