You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kylin.apache.org by "Shaofeng SHI (JIRA)" <ji...@apache.org> on 2017/01/15 14:47:26 UTC

[jira] [Commented] (KYLIN-2390) Wrong argument order for WinAggResetContextImpl()

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

Shaofeng SHI commented on KYLIN-2390:
-------------------------------------

Thanks Ted! [~mahongbin] could you please check this?

> Wrong argument order for WinAggResetContextImpl()
> -------------------------------------------------
>
>                 Key: KYLIN-2390
>                 URL: https://issues.apache.org/jira/browse/KYLIN-2390
>             Project: Kylin
>          Issue Type: Bug
>            Reporter: Ted Yu
>
> In EnumerableWindow#implement() , the order of last two parameters is swapped:
> {code}
>                         new WinAggResetContextImpl(builder6, agg.state, i_, startX, endX,
>                                 hasRows, partitionRowCount, frameRowCount));
> {code}



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