mkou commented on PR #2973:
URL: https://github.com/apache/calcite/pull/2973#issuecomment-1356124354
Hey Julian,
Yes we agree this is what I meant
>
> Le 16 déc. 2022 à 18:52, Julian Hyde ***@***.***> a écrit :
>
>
> @mkou, In SqlLibraryOperators, the TIMESTAMP function should return a
Calcite TIMESTAMP, DATE will return DATE, and there is no DATETIME function.
>
> Elsewhere (or in non-open source code), there can be a function called
DATETIME that returns a Calcite TIMESTAMP (what BQ calls a DATETIME), and a
function called TIMESTAMP that returns a Calcite TIMESTAMP WITH LOCAL TIME ZONE
(what BQ calls a TIMESTAMP). The DATETIME function would in fact be
SqlLibraryOperators.TIMESTAMP.withName("DATETIME"). And so forth.
>
> Sorry it's complicated. But we need to keep the function library
orthogonal to any type aliasing.
>
> —
> Reply to this email directly, view it on GitHub, or unsubscribe.
> You are receiving this because you were mentioned.
--
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]
