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 2011/05/04 19:35:46 UTC
svn commit: r1099535 - in /lucene/dev/branches/branch_3x: ./ lucene/
lucene/backwards/
lucene/src/test/org/apache/lucene/index/TestIndexWriter.java solr/
Author: mikemccand
Date: Wed May 4 17:35:45 2011
New Revision: 1099535
URL: http://svn.apache.org/viewvc?rev=1099535&view=rev
Log:
don't call Thread.interrupt after telling sub-thread to finish; it can cause false failure (merged from trunk)
Modified:
lucene/dev/branches/branch_3x/ (props changed)
lucene/dev/branches/branch_3x/lucene/ (props changed)
lucene/dev/branches/branch_3x/lucene/backwards/ (props changed)
lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
lucene/dev/branches/branch_3x/solr/ (props changed)
Modified: lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java?rev=1099535&r1=1099534&r2=1099535&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java (original)
+++ lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java Wed May 4 17:35:45 2011
@@ -2337,7 +2337,7 @@ public class TestIndexWriter extends Luc
volatile boolean failed;
volatile boolean finish;
- boolean allowInterrupt = false;
+ volatile boolean allowInterrupt = false;
@Override
public void run() {
@@ -2450,7 +2450,6 @@ public class TestIndexWriter extends Luc
}
}
t.finish = true;
- t.interrupt();
t.join();
assertFalse(t.failed);
}