This is an automated email from the ASF dual-hosted git repository.

jhyde pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/master by this push:
     new dd9bc94  Fix SQL example in comment
dd9bc94 is described below

commit dd9bc946483ce1d41d0552528098ae3a25500fec
Author: arnkore <[email protected]>
AuthorDate: Thu Oct 17 18:18:52 2019 +0800

    Fix SQL example in comment
    
    Close apache/calcite#1515
---
 .../main/java/org/apache/calcite/rel/rules/SubQueryRemoveRule.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/core/src/main/java/org/apache/calcite/rel/rules/SubQueryRemoveRule.java 
b/core/src/main/java/org/apache/calcite/rel/rules/SubQueryRemoveRule.java
index e71ffe6..347a880 100644
--- a/core/src/main/java/org/apache/calcite/rel/rules/SubQueryRemoveRule.java
+++ b/core/src/main/java/org/apache/calcite/rel/rules/SubQueryRemoveRule.java
@@ -235,9 +235,9 @@ public abstract class SubQueryRemoveRule extends RelOptRule 
{
       //   end as v
       // from emp as e
       // left outer join (
-      //   select max(deptno) as m, count(*) as c, count(deptno) as d,
+      //   select name, max(deptno) as m, count(*) as c, count(deptno) as d,
       //       "alwaysTrue" as indicator
-      //   group by name from emp) as q on e.name = q.name
+      //   from emp group by name) as q on e.name = q.name
       builder.push(e.rel)
           .aggregate(builder.groupKey(),
               builder.aggregateCall(minMax, builder.field(0)).as("m"),

Reply via email to