You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Sean Hsuan-Yi Chu (JIRA)" <ji...@apache.org> on 2015/09/18 00:28:04 UTC

[jira] [Commented] (CALCITE-888) Having parenthesis around window would produce wrong result

    [ https://issues.apache.org/jira/browse/CALCITE-888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14804640#comment-14804640 ] 

Sean Hsuan-Yi Chu commented on CALCITE-888:
-------------------------------------------

Pull request:
https://github.com/apache/incubator-calcite/pull/134

[~julianhyde] Can you please help review? Thanks!

> Having parenthesis around window would produce wrong result
> -----------------------------------------------------------
>
>                 Key: CALCITE-888
>                 URL: https://issues.apache.org/jira/browse/CALCITE-888
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Sean Hsuan-Yi Chu
>            Assignee: Sean Hsuan-Yi Chu
>
> 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)