[CALCITE-1782] AggregateExpandDistinctAggregatesRule should work on Aggregate instead of LogicalAggregate

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.13.0
• Assignee: Haohui Mai
• Reporter: Haohui Mai
• Created: 09/May/17 23:48
• Updated: 26/Jun/17 08:29
• Resolved: 21/May/17 17:46

Description

AggregateExpandDistinctAggregatesRule expects the operator to be a subclass of LogicalAggregate:

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

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