svn commit: r1233708 - in /lucene/dev/trunk/lucene: CHANGES.txt src/test/org/apache/lucene/util/TestSentinelIntSet.java

[yo...@apache.org]

Author: yonik
Date: Fri Jan 20 01:41:54 2012
New Revision: 1233708

URL: http://svn.apache.org/viewvc?rev=1233708&view=rev
Log:
LUCENE-3711: add CHANGES entry

Modified:
    lucene/dev/trunk/lucene/CHANGES.txt
    lucene/dev/trunk/lucene/src/test/org/apache/lucene/util/TestSentinelIntSet.java

Modified: lucene/dev/trunk/lucene/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/CHANGES.txt?rev=1233708&r1=1233707&r2=1233708&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/CHANGES.txt (original)
+++ lucene/dev/trunk/lucene/CHANGES.txt Fri Jan 20 01:41:54 2012
@@ -825,6 +825,12 @@ Bug fixes
 * LUCENE-3605: don't sleep in a retry loop when trying to locate the
   segments_N file (Robert Muir, Mike McCandless)
 
+* LUCENE-3711: SentinelIntSet with a small initial size can go into
+  an infinite loop when expanded.  This can affect grouping using
+  TermAllGroupsCollector or TermAllGroupHeadsCollector if instantiated with a
+  non default small size. (Martijn van Groningen, yonik)
+
+
 Optimizations
 
 * LUCENE-3653: Improve concurrency in VirtualMethod and AttributeSource by

Modified: lucene/dev/trunk/lucene/src/test/org/apache/lucene/util/TestSentinelIntSet.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/test/org/apache/lucene/util/TestSentinelIntSet.java?rev=1233708&r1=1233707&r2=1233708&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/test/org/apache/lucene/util/TestSentinelIntSet.java (original)
+++ lucene/dev/trunk/lucene/src/test/org/apache/lucene/util/TestSentinelIntSet.java Fri Jan 20 01:41:54 2012
@@ -48,18 +48,6 @@ public class TestSentinelIntSet extends 
     assertEquals(24, set.rehashCount);
   }
   
-  @Test
-  public void testRehash() throws Exception {
-    SentinelIntSet set = new SentinelIntSet(3, -1);
-    set.put(1);
-    set.find(99);
-    set.put(2);
-    set.find(99);
-    set.put(3);
-    set.find(99);
-    set.put(4);
-    set.find(99);
-  }
 
   @Test
   public void testRandom() throws Exception {