[CALCITE-4817] Expand SubstitutionVisitor of Aggregate with max/min, which column is the group by list of target

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Component/s: core
• Labels: pull-request-available
• Assignee: Xurenhe
• Reporter: Xurenhe
• Created: 01/Oct/21 09:09
• Updated: 15/Oct/21 03:33
• Resolved: 15/Oct/21 03:33

Description

Now, SubstitutionVisitor support `Query with distinct aggregate on the column is matched to the group list existing in target`, as CALCITE-4374 describes.

But. It's fail, when query aggregate with max/min based on the column, which is in the group by list of target.

Materialized view recognition should also success.

//代码占位符
@Test void testQueryNoDistinctOptionalityAggCallColInTargetGroupBy1() {
  final String mv = ""
      + "select \"name\", \"deptno\" "
      + "from \"emps\" group by \"name\", \"deptno\"";
  final String query = ""
      + "select \"name\", min(\"deptno\")\n"
      + "from \"emps\" group by \"name\"";
  sql(mv, query).ok();
}

Issue Links

• relates to: CALCITE-4374 Support materialized view recognition when query distinct aggregate on target GROUP BY column
• links to: GitHub Pull Request #2556