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

binlijin commented on HBASE-17332:
----------------------------------

The UT failure is unrelated to this patch.



> Replace HashMap to Array for DataBlockEncoding.idToEncoding
> -----------------------------------------------------------
>
>                 Key: HBASE-17332
>                 URL: https://issues.apache.org/jira/browse/HBASE-17332
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 2.0.0
>            Reporter: binlijin
>            Assignee: binlijin
>             Fix For: 2.0.0
>
>         Attachments: DBE.idToEncoding.png, 
> DBE.idToEncoding_afterOptimize.png, HBASE-17332.V2.patch, 
> HBASE-17332.V3.patch, HBASE-17332.branch-1.V1.patch, HBASE-17332.patch
>
>
> Array is faster than HashMap also no need to autoboxing.
> I can see the DataBlockEncoding#getDataBlockEncoderById is a hot method if 
> enable encoding in my perf test(Random read, all data cache in 
> LruBlockCache). 



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

Reply via email to