You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2010/08/04 18:36:36 UTC
svn commit: r982323 -
/lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
Author: mikemccand
Date: Wed Aug 4 16:36:35 2010
New Revision: 982323
URL: http://svn.apache.org/viewvc?rev=982323&view=rev
Log:
don't call Thread.interrupt after telling sub-thread to finish; it can cause false failure
Modified:
lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
Modified: lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java?rev=982323&r1=982322&r2=982323&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java (original)
+++ lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java Wed Aug 4 16:36:35 2010
@@ -4315,7 +4315,7 @@ public class TestIndexWriter extends Luc
volatile boolean failed;
volatile boolean finish;
- boolean allowInterrupt = false;
+ volatile boolean allowInterrupt = false;
@Override
public void run() {
@@ -4439,7 +4439,6 @@ public class TestIndexWriter extends Luc
}
t.allowInterrupt = false;
t.finish = true;
- t.interrupt();
t.join();
assertFalse(t.failed);
}