Steve Carlin created CALCITE-7478:
-------------------------------------

             Summary: 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


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