[CALCITE-5864] getValueAs reports the wrong number of milliseconds for WEEK and the wrong number of months for QUARTER

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: core
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Jonathan A Sternberg
• Created: 19/Jul/23 22:32
• Updated: 30/Dec/24 03:38

Description

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.

Issue Links

• is related to: CALCITE-5495 Allow WEEK and QUARTER in INTERVAL literals
• links to: GitHub Pull Request #3329