You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Julian Hyde (JIRA)" <ji...@apache.org> on 2015/09/18 19:38:04 UTC

[jira] [Resolved] (CALCITE-888) Overlay window loses PARTITION BY list

     [ https://issues.apache.org/jira/browse/CALCITE-888?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Julian Hyde resolved CALCITE-888.
---------------------------------
       Resolution: Fixed
    Fix Version/s: 1.5.0-incubating

Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/aaac1d23.

> Overlay window loses PARTITION BY list
> --------------------------------------
>
>                 Key: CALCITE-888
>                 URL: https://issues.apache.org/jira/browse/CALCITE-888
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Sean Hsuan-Yi Chu
>            Assignee: Julian Hyde
>             Fix For: 1.5.0-incubating
>
>
> Having parenthesis around window would produce wrong result. For example,
> {code}
> select empno, sum(empno) over (w), count(*) over w
> from emp
> window w as (partition by empno order by empno)
> {code}
> Notice that the second select item has parenthesis in the window clause. This query would generate a wrong plan:
> {code}
> LogicalProject(EMPNO=[$0], $1=[$9], $2=[$10])
>   LogicalWindow(window#0=[window(partition {} order by [0] range between UNBOUNDED PRECEDING and CURRENT ROW aggs [SUM($0)])], window#1=[window(partition {0} order by [0] range between UNBOUNDED PRECEDING and CURRENT ROW aggs [COUNT()])])
>     LogicalTableScan(table=[[CATALOG, SALES, EMP]])
> {code}
> where we have two windows in the LogicalWindow.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)