[ 
https://issues.apache.org/jira/browse/CALCITE-7478?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Steve Carlin resolved CALCITE-7478.
-----------------------------------
    Resolution: Not A Problem

> StandardConvertletTable now always creates CHECKED_TIMES for interval
> ---------------------------------------------------------------------
>
>                 Key: CALCITE-7478
>                 URL: https://issues.apache.org/jira/browse/CALCITE-7478
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Steve Carlin
>            Priority: Major
>
> This is more of a question rather than a bug for [~mbudiu] 
> I've been running my code against 1.42-SNAPSHOT and yesterday I had a 
> regression in my tests.  I'm pretty sure this is due to CALCITE-7443, 
> specifically the line
> [https://github.com/apache/calcite/blob/main/core/src/main/java/org/apache/calcite/sql2rel/StandardConvertletTable.java#L555]
> I ran a very simple query for my database:
> {code:java}
> select (cast("1970-01-01" as timestamp) + interval 3 days){code}
> I ran it through the debugger, and I did not hit any line checking 
> SqlConformance.checkedArithmetic().  But I did hit the line in the 
> StandardConvertletTable that always sets the operator as CHECKED_TIMES now 
> rather than TIMES.  This caused some code of mine to break.
> Is this something that is always meant to be there regardless of the setting 
> in SqlConformance?  If so, I will change my code to handle it; it's a very 
> simple change.  I'm wondering if this will affect others as well.
> Thanks!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to