[ https://issues.apache.org/jira/browse/HBASE-10749?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13935842#comment-13935842 ]
stack commented on HBASE-10749: ------------------------------- +1 for 0.96 > CellComparator.compareStatic() compares type wrongly > ---------------------------------------------------- > > Key: HBASE-10749 > URL: https://issues.apache.org/jira/browse/HBASE-10749 > Project: HBase > Issue Type: Bug > Affects Versions: 0.98.1, 0.99.0, 0.96.1.1 > Reporter: ramkrishna.s.vasudevan > Assignee: ramkrishna.s.vasudevan > Priority: Critical > Fix For: 0.98.1, 0.99.0 > > Attachments: HBASE-10749.patch > > > CellComparator.compareStatic(Cell a, Cell b) compares type wrongly. > {code} > //type > c = (0xff & a.getTypeByte()) - (0xff & b.getTypeByte()); > if (c != 0) return c; > {code} > In the normal case it should be the other way > {code} > // Compare types. Let the delete types sort ahead of puts; i.e. types > // of higher numbers sort before those of lesser numbers. Maximum (255) > // appears ahead of everything, and minimum (0) appears after > // everything. > return (0xff & rtype) - (0xff & ltype); > {code} > Found this issue while replacing KVComparator to use CellComparator. > [~mcorgan] > Could you confirm this once? Currently PrefixTree code path uses this > compareStatic method. -- This message was sent by Atlassian JIRA (v6.2#6252)