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

Ted Yu commented on HBASE-7845:
-------------------------------

I ran the failed tests and they passed locally.
{code}
+        throw new IOException("Unexpected getFakedKey result, fakeKey:" +
+      fakeKey + ", firstKeyInBlock:" + firstKeyInBlock);
{code}
Indentation is off.
{code}
+    Bytes.equals(newKey, KeyValue.ROW_LENGTH_SIZE, newRowLength, 
expectedArray, 0,
+      expectedArray.length);
+    //verify: same with "row + family + qualifier", return rightKey directly
{code}
Style: please leave a blank line above the comment in test.
{code}
+    public byte[] getFakedKey(final byte[] leftKey, final byte[] rightKey) {
+      return Arrays.copyOf(rightKey, rightKey.length);
+    }
{code}
Add @Override to above method.


                
> optimize hfile index key
> ------------------------
>
>                 Key: HBASE-7845
>                 URL: https://issues.apache.org/jira/browse/HBASE-7845
>             Project: HBase
>          Issue Type: Improvement
>          Components: HFile
>    Affects Versions: 0.96.0
>            Reporter: Liang Xie
>            Assignee: Liang Xie
>             Fix For: 0.98.0
>
>         Attachments: HBASE-7845.txt, HBASE-7845-v2.txt, HBASE-7845-v3.txt, 
> HBASE-7845-v4.txt, HBASE-7845-v5.txt
>
>
> Leveldb uses ByteWiseComparatorImpl::FindShortestSeparator() & 
> FindShortSuccessor() to reduce index key size, it would be helpful under 
> special conditions.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to