[ 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)