You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by ju...@apache.org on 2011/09/30 07:44:14 UTC

svn commit: r1177516 - /incubator/kafka/trunk/core/src/main/scala/kafka/producer/DefaultPartitioner.scala

Author: junrao
Date: Fri Sep 30 05:44:13 2011
New Revision: 1177516

URL: http://svn.apache.org/viewvc?rev=1177516&view=rev
Log:
trival fix to make hash code positive; patched by Joel Koshy; reviewed by Jun Rao

Modified:
    incubator/kafka/trunk/core/src/main/scala/kafka/producer/DefaultPartitioner.scala

Modified: incubator/kafka/trunk/core/src/main/scala/kafka/producer/DefaultPartitioner.scala
URL: http://svn.apache.org/viewvc/incubator/kafka/trunk/core/src/main/scala/kafka/producer/DefaultPartitioner.scala?rev=1177516&r1=1177515&r2=1177516&view=diff
==============================================================================
--- incubator/kafka/trunk/core/src/main/scala/kafka/producer/DefaultPartitioner.scala (original)
+++ incubator/kafka/trunk/core/src/main/scala/kafka/producer/DefaultPartitioner.scala Fri Sep 30 05:44:13 2011
@@ -24,6 +24,6 @@ private[kafka] class DefaultPartitioner[
     if(key == null)
       random.nextInt(numPartitions)
     else
-      key.hashCode % numPartitions
+      math.abs(key.hashCode) % numPartitions
   }
 }