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

Jim Kellerman commented on HBASE-407:
-------------------------------------

Code review:

Unused imports, remove:
java.util.ArrayList
java.util.Set

Potential null pointer access at line 295

Missing javadoc for public methods at lines 303, 308

Line 629: unchecked conversion. Can this be fixed using Java generics?


> Client should cache region locations in an LRU structure
> --------------------------------------------------------
>
>                 Key: HBASE-407
>                 URL: https://issues.apache.org/jira/browse/HBASE-407
>             Project: Hadoop HBase
>          Issue Type: Improvement
>          Components: client
>            Reporter: Bryan Duxbury
>            Assignee: Bryan Duxbury
>         Attachments: 407.patch
>
>
> Instead of keeping the region locations cached client side in a TreeMap, we 
> should use an LRU mechanism to help manage memory more dynamically.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to