You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by ji...@apache.org on 2007/12/14 20:58:33 UTC

svn commit: r604281 - in /lucene/hadoop/trunk/src/contrib/hbase: CHANGES.txt src/java/org/onelab/filter/HashFunction.java

Author: jimk
Date: Fri Dec 14 11:58:33 2007
New Revision: 604281

URL: http://svn.apache.org/viewvc?rev=604281&view=rev
Log:
HADOOP-2414 Fix ArrayIndexOutOfBoundsException in bloom filters.

Modified:
    lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt
    lucene/hadoop/trunk/src/contrib/hbase/src/java/org/onelab/filter/HashFunction.java

Modified: lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt?rev=604281&r1=604280&r2=604281&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt (original)
+++ lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt Fri Dec 14 11:58:33 2007
@@ -80,6 +80,7 @@
    HADOOP-2417 Fix critical shutdown problem introduced by HADOOP-2338
    HADOOP-2418 Fix assertion failures in TestTableMapReduce, TestTableIndex,
                and TestTableJoinMapReduce
+   HADOOP-2414 Fix ArrayIndexOutOfBoundsException in bloom filters.
    
   IMPROVEMENTS
    HADOOP-2401 Add convenience put method that takes writable

Modified: lucene/hadoop/trunk/src/contrib/hbase/src/java/org/onelab/filter/HashFunction.java
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/contrib/hbase/src/java/org/onelab/filter/HashFunction.java?rev=604281&r1=604280&r2=604281&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/contrib/hbase/src/java/org/onelab/filter/HashFunction.java (original)
+++ lucene/hadoop/trunk/src/contrib/hbase/src/java/org/onelab/filter/HashFunction.java Fri Dec 14 11:58:33 2007
@@ -112,7 +112,7 @@
       }
       int[] result = new int[nbHash];
       for (int i = 0, initval = 0; i < nbHash; i++) {
-        initval = result[i] = Math.abs(JenkinsHash.hash(b, initval)) % maxValue;
+        initval = result[i] = Math.abs(JenkinsHash.hash(b, initval) % maxValue);
       }
       return result;
   }//end hash()