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 2016/06/13 08:44:31 UTC

lucene-solr:branch_6x: don't use MockDirectoryWrapper here: it can cause testThreadSafety thread starvation so test times out after 2 hours

Repository: lucene-solr
Updated Branches:
  refs/heads/branch_6x c67258694 -> dc65594f5


don't use MockDirectoryWrapper here: it can cause testThreadSafety thread starvation so test times out after 2 hours


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/dc65594f
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/dc65594f
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/dc65594f

Branch: refs/heads/branch_6x
Commit: dc65594f5372242d481e0c1fca2a3038a51ec1dc
Parents: c672586
Author: Mike McCandless <mi...@apache.org>
Authored: Mon Jun 13 04:42:11 2016 -0400
Committer: Mike McCandless <mi...@apache.org>
Committed: Mon Jun 13 04:44:00 2016 -0400

----------------------------------------------------------------------
 .../apache/lucene/store/TestHardLinkCopyDirectoryWrapper.java | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/dc65594f/lucene/misc/src/test/org/apache/lucene/store/TestHardLinkCopyDirectoryWrapper.java
----------------------------------------------------------------------
diff --git a/lucene/misc/src/test/org/apache/lucene/store/TestHardLinkCopyDirectoryWrapper.java b/lucene/misc/src/test/org/apache/lucene/store/TestHardLinkCopyDirectoryWrapper.java
index 388b2f0..953dc59 100644
--- a/lucene/misc/src/test/org/apache/lucene/store/TestHardLinkCopyDirectoryWrapper.java
+++ b/lucene/misc/src/test/org/apache/lucene/store/TestHardLinkCopyDirectoryWrapper.java
@@ -30,7 +30,12 @@ public class TestHardLinkCopyDirectoryWrapper extends BaseDirectoryTestCase {
 
   @Override
   protected Directory getDirectory(Path file) throws IOException {
-    Directory open = random().nextBoolean() ? newFSDirectory(file) : newDirectory();
+    Directory open;
+    if (random().nextBoolean()) {
+      open = new RAMDirectory();
+    } else {
+      open = FSDirectory.open(file);
+    }
     return new HardlinkCopyDirectoryWrapper(open);
   }