Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Sat, 8 Sep 2012 10:51:08 +1100 (NCT)
From: "Ted Yu (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <748851470.53247.1347061868097.JavaMail.jiratomcat@arcas>
In-Reply-To: <1607302559.2232.1345838331263.JavaMail.jiratomcat@arcas>
Subject: [jira] [Commented] (HBASE-6658) Rename WritableByteArrayComparable
 to something not mentioning Writable
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Ted Yu commented on HBASE-6658:
-------------------------------

See http://docs.oracle.com/javase/6/docs/api/java/util/Comparator.html

While Comparable defines compareTo() method which is not in Comparator interface.

I suggest keeping Comparable in class names and only dropping Writable.
                
> Rename WritableByteArrayComparable to something not mentioning Writable
> -----------------------------------------------------------------------
>
>                 Key: HBASE-6658
>                 URL: https://issues.apache.org/jira/browse/HBASE-6658
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Gregory Chanan
>            Assignee: Gregory Chanan
>            Priority: Minor
>             Fix For: 0.96.0
>
>         Attachments: HBASE-6658.patch
>
>
> After HBASE-6477, WritableByteArrayComparable will no longer be Writable, so should be renamed.
> Current idea is ByteArrayComparator (since all the derived classes are *Comparator not *Comparable), but I'm open to suggestions.

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