hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Anoop Sam John (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-11202) Cleanup on HRegion class
Date Wed, 21 May 2014 06:59:38 GMT

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

Anoop Sam John commented on HBASE-11202:

bq.This was added by Jeffrey Zhong it seems. Jeffrey, should we keep them? 
No not by Jeffrey.  This was added by the commit for the issue HBASE-9645.  [~stack] seems
u had done that commit.  Remembering why these 2 APIs added boss?

> Cleanup on HRegion class
> ------------------------
>                 Key: HBASE-11202
>                 URL: https://issues.apache.org/jira/browse/HBASE-11202
>             Project: HBase
>          Issue Type: Bug
>          Components: regionserver
>    Affects Versions: 0.99.0
>            Reporter: Nicolas Liochon
>            Assignee: Nicolas Liochon
>            Priority: Minor
>             Fix For: 0.99.0
>         Attachments: 11202.v1.patch
> This is mostly trivial stuff
>  - remove some methods not used
>  - typos
>  - remove some @param w/o any info
>  - change the code that uses deprecated methods
> The only non trivial change is when we get the store from a cell: instead of using the
map, we iterate on the key set. Likely, it would be better to hava a sorted array instead
of a Map, as the number of store is fixed.  Could be done in a later patch.

This message was sent by Atlassian JIRA

View raw message