Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
Date: Fri, 14 Mar 2014 10:29:47 +0000 (UTC)
From: "ramkrishna.s.vasudevan (JIRA)" <jira@apache.org>
To: dev@hbase.apache.org
Message-ID: <JIRA.12701445.1394792962091.68309.1394792987973@arcas>
In-Reply-To: <JIRA.12701445.1394792962091@arcas>
References: <JIRA.12701445.1394792962091@arcas>
Subject: [jira] [Created] (HBASE-10749) CellComparator.compareStatic()
 compares type wrongly
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

ramkrishna.s.vasudevan created HBASE-10749:
----------------------------------------------

             Summary: 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.96.1.1, 0.98.1, 0.99.0
            Reporter: ramkrishna.s.vasudevan
            Assignee: ramkrishna.s.vasudevan
            Priority: Critical
             Fix For: 0.98.1, 0.99.0


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)