finos/vuu

CalculatedColumn parsing error handling

Opened this issue · 0 comments

Currently if parsing of the calculated column definition fails, different part of the code (CalcutedColumnVisitor) handles it differently

  • return null
  • return NullCalculatedColumnClause
  • return ErrorClause

We should use ErrorClause going forward for all parsing errors

Also the Atlr grammer doesn't seem to include Null as a potential values. So NullCalculatedColumnClause maybe redundant for now.
There is separate issue raised for handling null in filter and calculated column #1344