njriasan commented on PR #3329: URL: https://github.com/apache/calcite/pull/3329#issuecomment-1663288681
@jsternberg Wanted to comment an issue with this approach I found in case you still care about this. Inside `RexLiteral` the `intervalString` method uses the typeName information directly instead of the actual qualifier in `getTimeUnits()`. As a result, if you try convert from Rex to SQL, you will accidentally multiply the underlying value (by 7 for week and 3 for quarter). -- 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]
