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 2020/08/03 20:17:14 UTC
[hbase] branch branch-2.3 updated: HBASE-24808 skip empty log
cleaner delegate class names (WAS => cleaner.CleanerChore: Can NOT create
CleanerDelegate= ClassNotFoundException) (#2181)
This is an automated email from the ASF dual-hosted git repository.
stack pushed a commit to branch branch-2.3
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2.3 by this push:
new e57bfcb HBASE-24808 skip empty log cleaner delegate class names (WAS => cleaner.CleanerChore: Can NOT create CleanerDelegate= ClassNotFoundException) (#2181)
e57bfcb is described below
commit e57bfcba2b9f96107315f7ae4ce6c787a5111453
Author: Michael Stack <sa...@users.noreply.github.com>
AuthorDate: Mon Aug 3 13:15:19 2020 -0700
HBASE-24808 skip empty log cleaner delegate class names (WAS => cleaner.CleanerChore: Can NOT create CleanerDelegate= ClassNotFoundException) (#2181)
Signed-off-by: Anoop Sam John <an...@apache.org>
Signed-off-by: Viraj Jasani <vj...@apache.org>
Signed-off-by: Sean Busbey <bu...@apache.org>
---
.../java/org/apache/hadoop/hbase/master/cleaner/CleanerChore.java | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/cleaner/CleanerChore.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/cleaner/CleanerChore.java
index 7bac2e1..4331d49 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/cleaner/CleanerChore.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/cleaner/CleanerChore.java
@@ -151,7 +151,11 @@ public abstract class CleanerChore<T extends FileCleanerDelegate> extends Schedu
this.cleanersChain = new LinkedList<>();
String[] logCleaners = conf.getStrings(confKey);
if (logCleaners != null) {
- for (String className : logCleaners) {
+ for (String className: logCleaners) {
+ className = className.trim();
+ if (className.isEmpty()) {
+ continue;
+ }
T logCleaner = newFileCleaner(className, conf);
if (logCleaner != null) {
LOG.info("Initialize cleaner={}", className);