[ 
https://issues.apache.org/jira/browse/DRILL-3962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18031153#comment-18031153
 ] 

ASF GitHub Bot commented on DRILL-3962:
---------------------------------------

cgivre commented on code in PR #3026:
URL: https://github.com/apache/drill/pull/3026#discussion_r2445042315


##########
exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillAggregateExpandGroupingSetsRule.java:
##########


Review Comment:
   @rymarm Thanks for the review.  I updated the method per your 
recommendation.  I also added additional documentation and cleaned up the tests 
a bit. 





> Add support of ROLLUP, CUBE, GROUPING SETS, GROUPING, GROUPING_ID, GROUP_ID 
> support
> -----------------------------------------------------------------------------------
>
>                 Key: DRILL-3962
>                 URL: https://issues.apache.org/jira/browse/DRILL-3962
>             Project: Apache Drill
>          Issue Type: New Feature
>            Reporter: Jinfeng Ni
>            Assignee: Charles Givre
>            Priority: Major
>
> These functions are important for BI analytical workload.  Currently, Calcite 
> supports those functions, but neither the planning or execution in Drill 
> supports those functions. 
> DRILL-3802 blocks those functions in Drill planning. But we should provide 
> the support for those functions in both planning and execution of Drill. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to