You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ec...@apache.org on 2015/10/22 21:10:27 UTC
hbase git commit: HBASE-14658 Addendum;
Allow loading a MonkeyFactory by class name
Repository: hbase
Updated Branches:
refs/heads/branch-1.2 51c25ff58 -> 15777f1b0
HBASE-14658 Addendum; Allow loading a MonkeyFactory by class name
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/15777f1b
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/15777f1b
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/15777f1b
Branch: refs/heads/branch-1.2
Commit: 15777f1b04c23f10fd2c0f8fa64c1cee749cac0d
Parents: 51c25ff
Author: Elliott Clark <ec...@apache.org>
Authored: Thu Oct 22 11:22:53 2015 -0700
Committer: Elliott Clark <ec...@apache.org>
Committed: Thu Oct 22 12:10:23 2015 -0700
----------------------------------------------------------------------
.../apache/hadoop/hbase/chaos/factories/MonkeyFactory.java | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/15777f1b/hbase-it/src/test/java/org/apache/hadoop/hbase/chaos/factories/MonkeyFactory.java
----------------------------------------------------------------------
diff --git a/hbase-it/src/test/java/org/apache/hadoop/hbase/chaos/factories/MonkeyFactory.java b/hbase-it/src/test/java/org/apache/hadoop/hbase/chaos/factories/MonkeyFactory.java
index 2e5fd7b..9eb4eb3 100644
--- a/hbase-it/src/test/java/org/apache/hadoop/hbase/chaos/factories/MonkeyFactory.java
+++ b/hbase-it/src/test/java/org/apache/hadoop/hbase/chaos/factories/MonkeyFactory.java
@@ -89,13 +89,16 @@ public abstract class MonkeyFactory {
public static MonkeyFactory getFactory(String factoryName) {
MonkeyFactory fact = FACTORIES.get(factoryName);
- if (fact == null) {
+ if (fact == null && factoryName != null && !factoryName.isEmpty()) {
Class klass = null;
try {
klass = Class.forName(factoryName);
- fact = (MonkeyFactory) ReflectionUtils.newInstance(klass);
- } catch (ClassNotFoundException e) {
+ if (klass != null) {
+ fact = (MonkeyFactory) ReflectionUtils.newInstance(klass);
+ }
+ } catch (Exception e) {
LOG.error("Error trying to create " + factoryName + " could not load it by class name");
+ return null;
}
}
return fact;