[
https://issues.apache.org/jira/browse/HBASE-18251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16129337#comment-16129337
]
Hudson commented on HBASE-18251:
--------------------------------
FAILURE: Integrated in Jenkins build HBase-1.5 #9 (See
[https://builds.apache.org/job/HBase-1.5/9/])
HBASE-18251 Remove unnecessary traversing to the first and last keys in
(apurtell: rev 6255dc70014ad3faf70ae43a786e8d4f5b79dd41)
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CellSkipListSet.java
> Remove unnecessary traversing to the first and last keys in the CellSet
> -----------------------------------------------------------------------
>
> Key: HBASE-18251
> URL: https://issues.apache.org/jira/browse/HBASE-18251
> Project: HBase
> Issue Type: Improvement
> Affects Versions: 2.0.0-alpha-1
> Reporter: Anastasia Braginsky
> Assignee: Toshihiro Suzuki
> Fix For: 3.0.0, 1.4.0, 2.0.0-alpha-2
>
> Attachments: HBASE-18251.patch, HBASE-18251-v2.patch,
> HBASE-18251-v2.patch
>
>
> The implementation of finding the first and last keys in the CellSet is as
> following:
> {code}
> public Cell first() {
> return this.delegatee.get(this.delegatee.firstKey());
> }
> public Cell last() {
> return this.delegatee.get(this.delegatee.lastKey());
> }
> {code}
> Recall we have Cell to Cell mapping, therefore the methods bringing the
> first/last key, which allready return Cell. Thus no need to waist time on the
> get() method for the same Cell.
> Fix: return just the first/lastKey(), should be at least twice more effective.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
