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)