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

Lars Hofhansl updated HBASE-6711:
---------------------------------

    Attachment: 6711-0.96-v1.txt

Here's a simple patch for that.
The performance implications will typically be negligible here (except for 
extreme cases), but it reads better.
                
> Avoid local results copy in StoreScanner
> ----------------------------------------
>
>                 Key: HBASE-6711
>                 URL: https://issues.apache.org/jira/browse/HBASE-6711
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Lars Hofhansl
>            Assignee: Lars Hofhansl
>            Priority: Minor
>             Fix For: 0.96.0, 0.94.2
>
>         Attachments: 6711-0.96-v1.txt
>
>
> In StoreScanner the number of results is limited to avoid OOMs.
> However, this is done by first adding the KV into a local ArrayList and then 
> copying the entries in this list to the final result list.
> In turns out the this temporary list is only used to keep track of the size 
> of the result set in this loop. Can use a simple int instead.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to