julianhyde commented on a change in pull request #2458:
URL: https://github.com/apache/calcite/pull/2458#discussion_r675923009
##########
File path: core/src/main/java/org/apache/calcite/tools/RelBuilder.java
##########
@@ -934,7 +934,15 @@ public GroupKey groupKey(ImmutableBitSet groupSet) {
*
* <p>This method of creating a group key does not allow you to group on new
* expressions, only column projections, but is efficient, especially when
you
- * are coming from an existing {@link Aggregate}. */
+ * are coming from an existing {@link Aggregate}.
+ *
+ * <p>This method of creating a group key allow you to create groupKey be a
Review comment:
Yes. I've modified the javadoc with an example based on yours.
--
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]
