[ https://issues.apache.org/jira/browse/HBASE-5997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13409794#comment-13409794 ]
stack commented on HBASE-5997: ------------------------------ @Anoop Great work. Agree on the seekBefore finding. Lets fix that in another issue. On the patch, minor comment only (I'm good w/ commit): {code} if (firstKey == null) { {code} Is it possible if the file is empty say that we'll seek on every invocation of getFirstKey? This patch does not do you your compare of row only rather than compare of full key. Is it supposed to? > Fix concerns raised in HBASE-5922 related to HalfStoreFileReader > ---------------------------------------------------------------- > > Key: HBASE-5997 > URL: https://issues.apache.org/jira/browse/HBASE-5997 > Project: HBase > Issue Type: Bug > Affects Versions: 0.90.6, 0.92.1, 0.94.0, 0.96.0 > Reporter: ramkrishna.s.vasudevan > Assignee: Anoop Sam John > Fix For: 0.94.1 > > Attachments: HBASE-5997_0.94.patch, HBASE-5997_94 V2.patch, > Testcase.patch.txt > > > Pls refer to the comment > https://issues.apache.org/jira/browse/HBASE-5922?focusedCommentId=13269346&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13269346. > Raised this issue to solve that comment. Just incase we don't forget it. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira