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

Jim Kellerman commented on HBASE-557:
-------------------------------------

Nits only. Fix these and +1

HConnectionManager:
- unused imports: SortedMap, TreeMap, ImmutableBytesWritable, HbaseMapWritable, 
hadoop.io.Writable, HStoreKey

HTable:
- unused imports: HBbaseMapWritable, ImmutableBytesWritable

HRegionInterface:
- unused import HbaseMapWritable

> HTable.getRow() should receive RowResult objects
> ------------------------------------------------
>
>                 Key: HBASE-557
>                 URL: https://issues.apache.org/jira/browse/HBASE-557
>             Project: Hadoop HBase
>          Issue Type: Improvement
>            Reporter: Bryan Duxbury
>            Assignee: Bryan Duxbury
>            Priority: Minor
>             Fix For: 0.2.0
>
>         Attachments: 557-v2.patch, 557.patch
>
>
> Currently still uses HBaseMapWritable of HStoreKeys and Cells. HStoreKeys 
> will have a lot of redundant data (rows and timestamps), and we have to do a 
> transformation when it gets to the client to make it a SortedMap<Text, Cell>. 
> Take advantage of the RowResult class to greatly simplify. Maybe getRow 
> should just return a straight RowResult?

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