mihaibudiu commented on code in PR #4025:
URL: https://github.com/apache/calcite/pull/4025#discussion_r1823713674
##########
core/src/test/java/org/apache/calcite/rel/rel2sql/RelToSqlConverterTest.java:
##########
@@ -8741,6 +8741,48 @@ private void checkLiteral2(String expression, String
expected) {
.dialect(MssqlSqlDialect.DEFAULT).ok(mssqlExpected);
}
+ /** Test case for
+ * <a
href="https://issues.apache.org/jira/browse/CALCITE-6655";>[CALCITE-6655]
+ * Aggregation of deeply nested aggregations </a>.
Review Comment:
extra space before closing tag
You need to use the exact same message as in the JIRA case.
Please use this for your PR as well.
These conventions help us keep track of the correspondence between commits,
PRs, and JIRA cases.
--
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]
