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"),