[jira] [Created] (CALCITE-5864) getValueAs reports the wrong number of milliseconds for QUARTER and WEEK

["Jonathan A Sternberg (Jira)" <ji...@apache.org>]

Jonathan A Sternberg created CALCITE-5864:
---------------------------------------------

             Summary: getValueAs reports the wrong number of milliseconds for QUARTER and WEEK
                 Key: CALCITE-5864
                 URL: https://issues.apache.org/jira/browse/CALCITE-5864
             Project: Calcite
          Issue Type: Bug
          Components: core
            Reporter: Jonathan A Sternberg


The change that introduced WEEK and QUARTER processing here did not correctly implement the `evaluateIntervalLiteralAsX` functions for these two time units.

For both of them, it used the `fillIntervalValueArray` with the week parameter being used for months and the quarter parameter being used for months with no modifications to the underlying values.

For weeks, this results in the field being misused as the hour parameter so 2 weeks becomes 2 hours. For quarters, this results in the field being misused as months so 2 quarters becomes 2 months.

I believe the proper way to implement these is to perform the modification so weeks gets translated to 7 days and quarters gets translated to 3 months.

This seems to only affect the string versions such as `INTERVAL '2' WEEK`. The integer variants don't seem to use the literal pathway so they aren't affected.



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