You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by jh...@apache.org on 2019/10/17 20:27:54 UTC

[calcite] branch master updated: Fix SQL example in comment

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 <lu...@gmail.com>
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"),