[
https://issues.apache.org/jira/browse/HBASE-11202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14003141#comment-14003141
]
Nicolas Liochon commented on HBASE-11202:
-----------------------------------------
bq. +1 (if hadoopqa passes or if tests pass for you locally)
Tests are in progress, it seems ok so far...
bq. You mean "Does not do"?
yes, will fix on commit.
bq. Is this removal ok? The comment itself says that it is intended to be used
from CPs. So what if someone used this already? I dont know which Jira issue
added these 2 public methods.
Hum.. Good point. I did check, but found the wrong info. This was added by
[~jeffreyz] it seems. Jeffrey, should we keep them?
> 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
(v6.2#6252)
