[
https://issues.apache.org/jira/browse/CALCITE-1984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16164202#comment-16164202
]
Julian Hyde commented on CALCITE-1984:
--------------------------------------
The rewrite looks valid to me, because empid is already unique for any given
deptno value, due to the group by on the input.
If you disagree, see if you can get the query to produce a different result
than say postgreSQL or oracle.
> MaterialView rewrite failed
> ---------------------------
>
> Key: CALCITE-1984
> URL: https://issues.apache.org/jira/browse/CALCITE-1984
> Project: Calcite
> Issue Type: Bug
> Components: core
> Affects Versions: 1.13.0
> Reporter: fei.chen
> Assignee: Julian Hyde
>
> I had written a test as follows . It runs to succeed . But , the result seems
> not right . The “count distinct” aggregation should not be changed to “count”.
> {code:java}
> @Test public void testAggregateMaterializationOnCountDistinctQuery1() {
> checkMaterialize(
> "select \"deptno\", \"empid\" ,\"salary\"\n"
> + "from \"emps\" group by \"deptno\", \"empid\",\"salary\"",
> "select \"deptno\", count( distinct \"empid\")" + "from (select
> \"deptno\", \"empid\" \n"
> + "from \"emps\" group by \"deptno\", \"empid\") group by
> \"deptno\"",
> HR_FKUK_MODEL,
> CalciteAssert.checkResultContains(
> "EnumerableAggregate(group=[{0}], S=[COUNT($1)])\n" +
> " EnumerableTableScan(table=[[hr, m0]]"));
> }
> {code}
>
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
