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

Posted to issues@calcite.apache.org by "Jesus Camacho Rodriguez (JIRA)" <ji...@apache.org> on 2016/09/29 13:00:26 UTC

[jira] [Resolved] (CALCITE-1400) AggregatePullUpConstantsRule might adjust aggregation function parameters indices wrongly

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

Jesus Camacho Rodriguez resolved CALCITE-1400.
----------------------------------------------
       Resolution: Fixed
    Fix Version/s: 1.10.0

Thanks [~julianhyde].

Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/22ddc82 (including the test).

> AggregatePullUpConstantsRule might adjust aggregation function parameters indices wrongly
> -----------------------------------------------------------------------------------------
>
>                 Key: CALCITE-1400
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1400
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.9.0
>            Reporter: Jesus Camacho Rodriguez
>            Assignee: Jesus Camacho Rodriguez
>             Fix For: 1.10.0
>
>
> I explored a bit and I think this is a piece missing in CALCITE-1038. Introducing a Project operator below as part of the rewriting was removed, and thus, there is no need to adjust the indices of the aggregation function parameters. However, the code is still there.



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