[hbase] branch branch-2 updated: HBASE-21467 Fix flaky test TestCoprocessorClassLoader.testCleanupOldJars

[st...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

stack pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new b585dc4  HBASE-21467 Fix flaky test TestCoprocessorClassLoader.testCleanupOldJars
b585dc4 is described below

commit b585dc42ba20eb50b0e815596cc2305e6ca6c0d8
Author: OrDTesters <or...@gmail.com>
AuthorDate: Sun Nov 11 13:13:44 2018 -0600

    HBASE-21467 Fix flaky test TestCoprocessorClassLoader.testCleanupOldJars
---
 .../java/org/apache/hadoop/hbase/util/TestCoprocessorClassLoader.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestCoprocessorClassLoader.java b/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestCoprocessorClassLoader.java
index 94cf37d..f32972f 100644
--- a/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestCoprocessorClassLoader.java
+++ b/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestCoprocessorClassLoader.java
@@ -62,11 +62,12 @@ public class TestCoprocessorClassLoader {
     File tmpJarFile = new File(jarFile.getParent(), "/tmp/" + className + ".test.jar");
     if (tmpJarFile.exists()) tmpJarFile.delete();
     assertFalse("tmp jar file should not exist", tmpJarFile.exists());
+    ClassLoader parent = TestCoprocessorClassLoader.class.getClassLoader();
+    CoprocessorClassLoader.getClassLoader(new Path(jarFile.getParent()), parent, "112", conf);
     IOUtils.copyBytes(new FileInputStream(jarFile),
       new FileOutputStream(tmpJarFile), conf, true);
     assertTrue("tmp jar file should be created", tmpJarFile.exists());
     Path path = new Path(jarFile.getAbsolutePath());
-    ClassLoader parent = TestCoprocessorClassLoader.class.getClassLoader();
     CoprocessorClassLoader.parentDirLockSet.clear(); // So that clean up can be triggered
     ClassLoader classLoader = CoprocessorClassLoader.getClassLoader(path, parent, "111", conf);
     assertNotNull("Classloader should be created", classLoader);