You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2014/03/29 04:35:08 UTC

svn commit: r1582945 - /hbase/branches/0.98/hbase-common/src/test/java/org/apache/hadoop/hbase/ClassFinder.java

Author: stack
Date: Sat Mar 29 03:35:08 2014
New Revision: 1582945

URL: http://svn.apache.org/r1582945
Log:
HBASE-9120 ClassFinder logs errors that are not

Modified:
    hbase/branches/0.98/hbase-common/src/test/java/org/apache/hadoop/hbase/ClassFinder.java

Modified: hbase/branches/0.98/hbase-common/src/test/java/org/apache/hadoop/hbase/ClassFinder.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.98/hbase-common/src/test/java/org/apache/hadoop/hbase/ClassFinder.java?rev=1582945&r1=1582944&r2=1582945&view=diff
==============================================================================
--- hbase/branches/0.98/hbase-common/src/test/java/org/apache/hadoop/hbase/ClassFinder.java (original)
+++ hbase/branches/0.98/hbase-common/src/test/java/org/apache/hadoop/hbase/ClassFinder.java Sat Mar 29 03:35:08 2014
@@ -133,7 +133,7 @@ public class ClassFinder {
     try {
       jarFile = new JarInputStream(new FileInputStream(jarFileName));
     } catch (IOException ioEx) {
-      LOG.error("Failed to look for classes in " + jarFileName + ": " + ioEx);
+      LOG.warn("Failed to look for classes in " + jarFileName + ": " + ioEx);
       throw ioEx;
     }
 
@@ -147,7 +147,7 @@ public class ClassFinder {
           if (!proceedOnExceptions) {
             throw ioEx;
           }
-          LOG.error("Failed to get next entry from " + jarFileName + ": " + ioEx);
+          LOG.warn("Failed to get next entry from " + jarFileName + ": " + ioEx);
           break;
         }
         if (entry == null) {
@@ -172,7 +172,7 @@ public class ClassFinder {
         Class<?> c = makeClass(className, proceedOnExceptions);
         if (c != null) {
           if (!classes.add(c)) {
-            LOG.error("Ignoring duplicate class " + className);
+            LOG.warn("Ignoring duplicate class " + className);
           }
         }
       }
@@ -186,13 +186,13 @@ public class ClassFinder {
       boolean proceedOnExceptions) throws ClassNotFoundException, LinkageError {
     Set<Class<?>> classes = new HashSet<Class<?>>();
     if (!baseDirectory.exists()) {
-      LOG.error("Failed to find " + baseDirectory.getAbsolutePath());
+      LOG.warn("Failed to find " + baseDirectory.getAbsolutePath());
       return classes;
     }
 
     File[] files = baseDirectory.listFiles(this.fileFilter);
     if (files == null) {
-      LOG.error("Failed to get files from " + baseDirectory.getAbsolutePath());
+      LOG.warn("Failed to get files from " + baseDirectory.getAbsolutePath());
       return classes;
     }
 
@@ -207,7 +207,7 @@ public class ClassFinder {
         Class<?> c = makeClass(className, proceedOnExceptions);
         if (c != null) {
           if (!classes.add(c)) {
-            LOG.error("Ignoring duplicate class " + className);
+            LOG.warn("Ignoring duplicate class " + className);
           }
         }
       }