beyond1920 commented on a change in pull request #2524:
URL: https://github.com/apache/calcite/pull/2524#discussion_r716130554
##########
File path:
core/src/main/java/org/apache/calcite/sql2rel/StandardConvertletTable.java
##########
@@ -753,6 +755,38 @@ public RexNode convertWindowFunction(
return cx.getRexBuilder().makeCall(returnType, fun, exprs);
}
+ public RexNode convertSessionWindowFunction(
+ SqlRexContext cx,
+ SqlSessionTableFunction fun,
+ SqlCall call) {
+ // The first operand of window function is actually a query, skip that.
+ final List<SqlNode> operands = Util.skip(call.getOperandList());
+ final List<RexNode> exprs =
+ convertOperands(cx, call, operands,
+ SqlOperandTypeChecker.Consistency.NONE);
+ RelDataType returnType =
Review comment:
Hi, Danny, drop the old DESCRIPTOR(partition_key) syntax is an option.
however it would cause compatibility problem, I'm not sure whether the user
could accept this uncompatible update.
--
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]
