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

Hanifi Gunes commented on DRILL-1539:
-------------------------------------

1. it sounds better to set lastPopulatedValueIndex to number `to`.

2. Agree that this should be a seperate JIRA.

I will verify 1 & 2

> RepeatedMapVector maintains incorrect last non-empty element index.
> -------------------------------------------------------------------
>
>                 Key: DRILL-1539
>                 URL: https://issues.apache.org/jira/browse/DRILL-1539
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Hanifi Gunes
>            Assignee: Jinfeng Ni
>         Attachments: DRILL-1539.1.patch.txt
>
>
> RepeatedMapVector maintains the last non-empty element index. The invariant 
> for the variable is that it must start from -1 and it must be less than the 
> group count. Current implementation violates this invariant. Drill will raise 
> IndexOutOfBounds exception for certain cases led by this violation.



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

Reply via email to