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

Julian Hyde commented on CALCITE-4634:
--------------------------------------

I think it is worthwhile to have two variants (i.e. a boolean flag in the 
Config). The current variant should remain the default.

Both variants should detect whether minRowCount >= 1D and if so, go ahead and 
remove the final group key.

> Improve AggregateProjectPullUpConstantsRule to remove all constant keys
> -----------------------------------------------------------------------
>
>                 Key: CALCITE-4634
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4634
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 1.16.0
>            Reporter: Yingyu
>            Priority: Major
>
> Optimizer rule *AggregateProjectPullUpConstantsRule* removes constant keys 
> from an Aggregate. However, this rules never removes the last column, because 
> Aggregate([]) returns 1 row even if its input is empty. 
> An enhancement could be done to AggregateProjectPullUpConstantsRule so that 
> it removes all constant keys:
>  - Remove all constant keys from an Aggregate
>  - (Only)When no key is left from previous step (Aggregate becomes 
> Aggregate([])), add a filter to the Aggregate([]) so that it returns empty 
> dataset if its input is empty. 
> E.g. 
> {noformat}
> SELECT 'abc' FROM table_name GROUP BY ('abc')
> {noformat}
> will be translated to:
> {noformat}
> SELECT 'abc' FROM table_name GROUP BY () having count(*)>0
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to