yashmayya commented on code in PR #14782:
URL: https://github.com/apache/pinot/pull/14782#discussion_r1908287186
##########
pinot-query-planner/src/main/java/org/apache/pinot/calcite/sql/fun/PinotOperatorTable.java:
##########
@@ -69,6 +72,30 @@ public static PinotOperatorTable instance() {
return INSTANCE.get();
}
+ // The standard Calcite + and - operators don't support operations on
TIMESTAMP types. However, Pinot supports these
+ // operations, so we need to define our own operators. Note that Postgres
supports - on TIMESTAMP types, but not +.
+ // Calcite only supports such operations if the second operand is an
interval (similar to Postgres for the +
+ // operator).
+ public static final SqlBinaryOperator PINOT_PLUS =
Review Comment:
I think that should be fixable by adding a new constructor to
`PinotSqlFunction` and allowing caller to pass in `SqlKind` though.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
