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

Posted to issues@calcite.apache.org by "Muhammad Gelbana (JIRA)" <ji...@apache.org> on 2018/06/04 20:47:00 UTC

[jira] [Comment Edited] (CALCITE-1436) AggregateNode NPE for aggregators other than SUM/COUNT

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

Muhammad Gelbana edited comment on CALCITE-1436 at 6/4/18 8:46 PM:
-------------------------------------------------------------------

All tests ran fine. I created a Pull request: [https://github.com/apache/calcite/pull/719]


was (Author: mgelbana):
All tests ran find. I created a Pull request: [https://github.com/apache/calcite/pull/719]

> AggregateNode NPE for aggregators other than SUM/COUNT
> ------------------------------------------------------
>
>                 Key: CALCITE-1436
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1436
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>            Reporter: Gian Merlino
>            Assignee: Julian Hyde
>            Priority: Major
>
> AggregateNode.getAccumulator does this for any aggregation other than COUNT or SUM:
>       final AggImpState agg = new AggImpState(0, call, false);
>       int stateSize = agg.state.size();
> This NPEs because "state" is null on freshly created AggImpState instances.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)