[CALCITE-2944] Deprecate Aggregate indicators and remove fields where possible

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.20.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Haisheng Yuan
• Created: 22/Mar/19 01:20
• Updated: 24/Jun/19 15:03
• Resolved: 01/Jun/19 23:35

Description

Although Aggregate.indicator is discouraged to use, but no test cases guarantee indicator can work correctly. I tried to set indicator to always false in Aggregate constructor, it passed all the tests.

Issue Links

• links to: GitHub Pull Request #1243