You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "Chris Westin (JIRA)" <ji...@apache.org> on 2015/04/22 20:46:59 UTC

[jira] [Updated] (DRILL-2261) Replace HashMap in IncomingBuffers with an array based data structure

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

Chris Westin updated DRILL-2261:
--------------------------------
    Fix Version/s:     (was: 1.0.0)
                   1.1.0

> Replace HashMap<Integer, DataCollector> in IncomingBuffers with an array based data structure
> ---------------------------------------------------------------------------------------------
>
>                 Key: DRILL-2261
>                 URL: https://issues.apache.org/jira/browse/DRILL-2261
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Execution - Flow
>    Affects Versions: 0.7.0
>            Reporter: Venki Korukanti
>            Assignee: Venki Korukanti
>            Priority: Minor
>             Fix For: 1.1.0
>
>
> Currently we have Map<Integer, DataCollector> which stores the mapping of major fragment id and DataCollector instance pairs. For every record batch arrived, we need to autobox primitive integer and hash. This is unnecessary cost, instead we could have a simpler array based structure like TIntArrayList (trove collection) or custom array based structure. Key value (in this case major fragment id) is between 0 and numStages - 1. numStages is not going to be a huge number. Utilizing this range we could either use TIntArrayList or custom array based structure.   



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