You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by da...@apache.org on 2018/12/11 09:28:09 UTC

[40/47] lucene-solr:jira/http2: SOLR-12801: Fix TestSolrCloudWithKerberosAlt.

SOLR-12801: Fix TestSolrCloudWithKerberosAlt.


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

Branch: refs/heads/jira/http2
Commit: d96475a0bb9e41b285c273e1b70d984208d6fd3a
Parents: b6cbb84
Author: markrmiller <ma...@apache.org>
Authored: Mon Dec 10 11:46:27 2018 -0600
Committer: markrmiller <ma...@apache.org>
Committed: Mon Dec 10 11:46:34 2018 -0600

----------------------------------------------------------------------
 .../core/src/test/org/apache/solr/cloud/KerberosTestServices.java | 3 +--
 .../test/org/apache/solr/cloud/TestSolrCloudWithKerberosAlt.java  | 3 +++
 2 files changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/d96475a0/solr/core/src/test/org/apache/solr/cloud/KerberosTestServices.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/KerberosTestServices.java b/solr/core/src/test/org/apache/solr/cloud/KerberosTestServices.java
index 4ec5640..cccf590 100644
--- a/solr/core/src/test/org/apache/solr/cloud/KerberosTestServices.java
+++ b/solr/core/src/test/org/apache/solr/cloud/KerberosTestServices.java
@@ -64,7 +64,6 @@ public class KerberosTestServices {
       Locale.setDefault(Locale.US);
     }
 
-    File dir = null;
     // There is time lag between selecting a port and trying to bind with it. It's possible that
     // another service captures the port in between which'll result in BindException.
     boolean bindException;
@@ -76,7 +75,7 @@ public class KerberosTestServices {
         kdc = getKdc(workDir);
         kdc.start();
       } catch (BindException e) {
-        FileUtils.deleteDirectory(dir); // clean directory
+        FileUtils.deleteDirectory(workDir); // clean directory
         numTries++;
         if (numTries == 3) {
           log.error("Failed setting up MiniKDC. Tried " + numTries + " times.");

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/d96475a0/solr/core/src/test/org/apache/solr/cloud/TestSolrCloudWithKerberosAlt.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/TestSolrCloudWithKerberosAlt.java b/solr/core/src/test/org/apache/solr/cloud/TestSolrCloudWithKerberosAlt.java
index 9d56204..0978e57 100644
--- a/solr/core/src/test/org/apache/solr/cloud/TestSolrCloudWithKerberosAlt.java
+++ b/solr/core/src/test/org/apache/solr/cloud/TestSolrCloudWithKerberosAlt.java
@@ -21,6 +21,8 @@ import java.lang.invoke.MethodHandles;
 import java.nio.charset.StandardCharsets;
 
 import com.carrotsearch.randomizedtesting.annotations.ThreadLeakFilters;
+import com.carrotsearch.randomizedtesting.annotations.ThreadLeakLingering;
+
 import org.apache.commons.io.FileUtils;
 import org.apache.lucene.util.Constants;
 import org.apache.lucene.util.LuceneTestCase;
@@ -46,6 +48,7 @@ import org.slf4j.LoggerFactory;
 })
 
 @LuceneTestCase.Slow
+@ThreadLeakLingering(linger = 10000) // minikdc has some lingering threads
 public class TestSolrCloudWithKerberosAlt extends SolrCloudTestCase {
 
   private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());