You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by vl...@apache.org on 2018/08/28 09:54:33 UTC

calcite git commit: [CALCITE-311] Added a test-case for filter after window aggregate

Repository: calcite
Updated Branches:
  refs/heads/master 141781bf4 -> 9f7e565ff


[CALCITE-311] Added a test-case for filter after window aggregate

Filters should not be pushed through window aggregate since it might impact results.


Project: http://git-wip-us.apache.org/repos/asf/calcite/repo
Commit: http://git-wip-us.apache.org/repos/asf/calcite/commit/9f7e565f
Tree: http://git-wip-us.apache.org/repos/asf/calcite/tree/9f7e565f
Diff: http://git-wip-us.apache.org/repos/asf/calcite/diff/9f7e565f

Branch: refs/heads/master
Commit: 9f7e565ffcadd161e8a92906a493f551b9c02f2b
Parents: 141781b
Author: Vladimir Sitnikov <si...@gmail.com>
Authored: Tue Aug 28 12:53:46 2018 +0300
Committer: Vladimir Sitnikov <si...@gmail.com>
Committed: Tue Aug 28 12:53:46 2018 +0300

----------------------------------------------------------------------
 core/src/test/resources/sql/winagg.iq | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/calcite/blob/9f7e565f/core/src/test/resources/sql/winagg.iq
----------------------------------------------------------------------
diff --git a/core/src/test/resources/sql/winagg.iq b/core/src/test/resources/sql/winagg.iq
index ff0aadc..c7b9482 100644
--- a/core/src/test/resources/sql/winagg.iq
+++ b/core/src/test/resources/sql/winagg.iq
@@ -404,6 +404,22 @@ order by gender, r;
 
 !ok
 
+# [CALCITE-311] Wrong results when filtering the results of windowed aggregation
+
+select * from (
+  select "empid", count(*) over () c
+    from "hr"."emps"
+) where "empid"=100;
+
++-------+---+
+| empid | C |
++-------+---+
+|   100 | 4 |
++-------+---+
+(1 row)
+
+!ok
+
 # [CALCITE-2081] Two windows under a JOIN
 select a."deptno", a.r as ar, b.r as br
 from (