You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by la...@apache.org on 2013/08/04 03:52:56 UTC
svn commit: r1510088 -
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/ClassFinder.java
Author: larsh
Date: Sun Aug 4 01:52:56 2013
New Revision: 1510088
URL: http://svn.apache.org/r1510088
Log:
HBASE-9120 ClassFinder logs errors that are not (Sergey)
Modified:
hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/ClassFinder.java
Modified: hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/ClassFinder.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/ClassFinder.java?rev=1510088&r1=1510087&r2=1510088&view=diff
==============================================================================
--- hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/ClassFinder.java (original)
+++ hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/ClassFinder.java Sun Aug 4 01:52:56 2013
@@ -120,7 +120,7 @@ public class ClassFinder {
if (!proceedOnExceptions) {
throw ioEx;
}
- LOG.error("Failed to look for classes in " + jarFileName + ": " + ioEx);
+ LOG.warn("Failed to look for classes in " + jarFileName + ": " + ioEx);
}
Set<Class<?>> classes = new HashSet<Class<?>>();
@@ -132,7 +132,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) {
@@ -156,7 +156,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);
}
}
}
@@ -167,13 +167,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;
}
@@ -188,7 +188,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);
}
}
}
@@ -205,12 +205,12 @@ public class ClassFinder {
if (!proceedOnExceptions) {
throw classNotFoundEx;
}
- LOG.error("Failed to instantiate or check " + className + ": " + classNotFoundEx);
+ LOG.info("Failed to instantiate or check " + className + ": " + classNotFoundEx);
} catch (LinkageError linkageEx) {
if (!proceedOnExceptions) {
throw linkageEx;
}
- LOG.error("Failed to instantiate or check " + className + ": " + linkageEx);
+ LOG.info("Failed to instantiate or check " + className + ": " + linkageEx);
}
return null;
}