[ 
https://issues.apache.org/jira/browse/HADOOP-5528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12689260#action_12689260
 ] 

Chris Douglas commented on HADOOP-5528:
---------------------------------------

* It would make sense for the deprecated partitioner class to extend the 
partitioner from mapreduce, per the idiom used in HADOOP-1230
* Code using BinaryPartitioner might be more readable if it had a static method 
for setting the left and right indices in the config. There are other examples 
in lib
* The indexer class hierarchy confuses me. Instead of making the semantics 
configurable, defining and describing them in the aforementioned static method 
would be much easier. I'd also be a fan of using the python semantics, all the 
time.

> Binary partitioner
> ------------------
>
>                 Key: HADOOP-5528
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5528
>             Project: Hadoop Core
>          Issue Type: New Feature
>          Components: mapred
>            Reporter: Klaas Bosteels
>            Assignee: Klaas Bosteels
>         Attachments: HADOOP-5528.patch, HADOOP-5528.patch, HADOOP-5528.patch
>
>
> It would be useful to have a {{BinaryPartitioner}} that partitions 
> {{BinaryComparable}} keys by hashing a configurable part of the bytes array 
> corresponding to each key.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to