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

Chenxi Li updated ARROW-12771:
------------------------------
    Description: 
The parameter of {{MaybeReserve}} in {{has_count}}'s {{Consume}} function is 
incorrect.
{code:java}
MaybeReserve(counts_.length(), batch, [&](int64_t added_groups)
{code}
The {{counts_.length()}} means the buffer's byte size, it should be replaced 
with {{num_groups_}}.  Otherwise, subsequent chunked arrays with more groups 
are skipped as {{counts_.length()}} is actually {{num_groups_*sizeof(int64_t)}}

  was:
The parameter of {{MaybeReserve}} in {{has_count'}}s Consume function is 
incorrect.
{code:java}
MaybeReserve(counts_.length(), batch, [&](int64_t added_groups)
{code}
The {{counts_.length()}} means the buffer's byte size, it should be replaced 
with {{num_groups_. }}Otherwise subsequent chunked arrays with more groups are 
skipped as {{counts_.length() }}is  actually {{num_groups_*sizeof(int64_t).}}


> [C++] Arrow compute hash_count skips following chunked arrays in streaming 
> execution 
> -------------------------------------------------------------------------------------
>
>                 Key: ARROW-12771
>                 URL: https://issues.apache.org/jira/browse/ARROW-12771
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>    Affects Versions: 4.0.0
>            Reporter: Chenxi Li
>            Priority: Major
>
> The parameter of {{MaybeReserve}} in {{has_count}}'s {{Consume}} function is 
> incorrect.
> {code:java}
> MaybeReserve(counts_.length(), batch, [&](int64_t added_groups)
> {code}
> The {{counts_.length()}} means the buffer's byte size, it should be replaced 
> with {{num_groups_}}.  Otherwise, subsequent chunked arrays with more groups 
> are skipped as {{counts_.length()}} is actually 
> {{num_groups_*sizeof(int64_t)}}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to