[PHOENIX-6822] Multiplier for Temporal round/floor/ceiling functions is not implemented and/or poorly defined

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Assignee: Unassigned
• Reporter: Istvan Toth
• Created: 27/Oct/22 08:01
• Updated: 29/Nov/22 20:13

Description

We are defining a multiplier parameter for the round/floor/ceil functions for temporal values.

https://phoenix.apache.org/language/functions.html#trunc

However, we only actually use that parameter for days, hours, minutes and seconds, but ignore them for weeks, months and years.

(weeks, months, and years are not mentioned in the documentation)

Even for days, and hours the implementation is very arbitrary, using the number of days/hours since epoch as a basis for the integer division function (as opposed to day of week, month, year etc. or hour of day)

 

Issue Links

• is caused by: PHOENIX-2433 support additional time units (like week/month/year) in Trunc() round() and Ceil()
• is related to: PHOENIX-5066 The TimeZone is incorrectly used during writing or reading data
• is related to: PHOENIX-6823 calling Joda-based round() function on temporal PK field causes division by zero error