jinxing64 commented on a change in pull request #1510: [CALCITE-3418] Grouped
Window Function TUMBLE should return "$TUMBLE"…
URL: https://github.com/apache/calcite/pull/1510#discussion_r339323642
##########
File path: core/src/test/java/org/apache/calcite/test/SqlValidatorTest.java
##########
@@ -10093,22 +10093,22 @@ private void checkCustomColumnResolving(String
table) {
+ "from orders\n"
+ "group by floor(rowtime to hour)")
.fails("Call to auxiliary group function 'TUMBLE_END' must have "
- + "matching call to group function 'TUMBLE' in GROUP BY clause");
+ + "matching call to group function '\\$TUMBLE' in GROUP BY
clause");
Review comment:
Do we need this change ?
It seems we are recommending to write the sql as `GROUP BY $TUMBLE(rowtime,
interval '2' hour, time '00:12:00')`
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
