[GitHub] [solr] thelabdude opened a new pull request #626: SOLR-15974: Remove Calcite's ENUMERABLE_AGGREGATE_RULE as Solr only supports push-down for LogicalAggregate

[GitBox <gi...@apache.org>]

thelabdude opened a new pull request #626:
URL: https://github.com/apache/solr/pull/626


   https://issues.apache.org/jira/browse/SOLR-15974
   
   Disable the `EnumerableRules.ENUMERABLE_AGGREGATE_RULE` optimization so that Solr only deals with LogicalAggregate. I looked into adding another rule for EnumerableAggregate but it seemed like we don't need this optimization from Calcite since we're pushing down into Solr's agg engine anyway, we don't need a bunch of optimizations in the Calcite layer.


-- 
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: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org

[GitHub] [solr] thelabdude merged pull request #626: SOLR-15974: Remove Calcite's ENUMERABLE_AGGREGATE_RULE as Solr only supports push-down for LogicalAggregate

[GitBox <gi...@apache.org>]

thelabdude merged pull request #626:
URL: https://github.com/apache/solr/pull/626


   


-- 
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: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org