You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@calcite.apache.org by "Haohui Mai (JIRA)" <ji...@apache.org> on 2017/05/09 23:49:04 UTC

[jira] [Created] (CALCITE-1782) AggregateExpandDistinctAggregatesRule should work on Aggregate instead of LogicalAggregate

Haohui Mai created CALCITE-1782:
-----------------------------------

             Summary: AggregateExpandDistinctAggregatesRule should work on Aggregate instead of LogicalAggregate
                 Key: CALCITE-1782
                 URL: https://issues.apache.org/jira/browse/CALCITE-1782
             Project: Calcite
          Issue Type: Bug
            Reporter: Haohui Mai
            Assignee: Haohui Mai


{{AggregateExpandDistinctAggregatesRule}} expects the operator to be a subclass of {{LogicalAggregate}}:

{noformat}
  public AggregateExpandDistinctAggregatesRule(
      Class<? extends LogicalAggregate> clazz,
      RelFactories.JoinFactory joinFactory) {
    this(clazz, false, RelBuilder.proto(Contexts.of(joinFactory)));
  }
{noformat}

However, {{LogicalAggregate}} is a final class which cannot be inherited. This jira proposes to change the signature from {{LogicalAggregate}} to {{Aggregate}}.





--
This message was sent by Atlassian JIRA
(v6.3.15#6346)