[
https://issues.apache.org/jira/browse/BEAM-12033?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kyle Weaver updated BEAM-12033:
-------------------------------
Description:
Numeric values in ZetaSQL are subject to constraints: they must fall within
certain hardcoded limits, and the scale also cannot exceed 9. Currently these
constraints are enforced when preparing the expression in BeamZetaSqlCalcRel.
However, to force BeamCalcRel to have the same behavior, we should check the
constraints earlier, during expression conversion.
https://github.com/google/zetasql/blob/697da738be2af3b6b9492c73c55616d8b9fed0b6/javatests/com/google/zetasql/ValueTest.java#L727-L745
was:
Numeric literals in ZetaSQL are subject to constraints: they must fall within
certain hardcoded limits, and the scale also cannot exceed 9. Currently these
constraints are enforced when preparing the expression in BeamZetaSqlCalcRel.
However, to force BeamCalcRel to have the same behavior, we should check the
constraints earlier, during expression conversion.
https://github.com/google/zetasql/blob/697da738be2af3b6b9492c73c55616d8b9fed0b6/javatests/com/google/zetasql/ValueTest.java#L727-L745
> Validate casts from double literals to numeric during expression conversion.
> ----------------------------------------------------------------------------
>
> Key: BEAM-12033
> URL: https://issues.apache.org/jira/browse/BEAM-12033
> Project: Beam
> Issue Type: Improvement
> Components: dsl-sql-zetasql
> Reporter: Kyle Weaver
> Assignee: Kyle Weaver
> Priority: P2
>
> Numeric values in ZetaSQL are subject to constraints: they must fall within
> certain hardcoded limits, and the scale also cannot exceed 9. Currently these
> constraints are enforced when preparing the expression in BeamZetaSqlCalcRel.
> However, to force BeamCalcRel to have the same behavior, we should check the
> constraints earlier, during expression conversion.
> https://github.com/google/zetasql/blob/697da738be2af3b6b9492c73c55616d8b9fed0b6/javatests/com/google/zetasql/ValueTest.java#L727-L745
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
