normanj-bitquill commented on PR #3794: URL: https://github.com/apache/calcite/pull/3794#issuecomment-2123546551
Since RedShift and PostgreSQL share the same implementation, they both now support strings or raw time units as the first argument. If a string is used for the first argument, `DAYOFWEEK`, `DAYOFYEAR` and `ISODOY` are not working due to limitations in the TimeFrames class. https://github.com/apache/calcite/blob/main/core/src/main/java/org/apache/calcite/rel/type/TimeFrames.java#L86 Validation of the string argument is done against the map generated in `TimeFrames`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
