You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by bu...@apache.org on 2019/11/15 02:42:14 UTC
[hbase] branch master updated: HBASE-23283 Provide clear and
consistent logging about the period of enabled chores
This is an automated email from the ASF dual-hosted git repository.
busbey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/master by this push:
new 73f3e1d HBASE-23283 Provide clear and consistent logging about the period of enabled chores
73f3e1d is described below
commit 73f3e1d4a4c15211bc85cc25ba8b99893fc26f24
Author: Mingliang Liu <li...@apache.org>
AuthorDate: Tue Nov 12 23:59:08 2019 -0800
HBASE-23283 Provide clear and consistent logging about the period of enabled chores
Signed-off-by: Sean Busbey <bu...@apache.org>
---
hbase-common/src/main/java/org/apache/hadoop/hbase/ChoreService.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/ChoreService.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/ChoreService.java
index 5e83ef3..0ad52b2 100644
--- a/hbase-common/src/main/java/org/apache/hadoop/hbase/ChoreService.java
+++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/ChoreService.java
@@ -151,9 +151,10 @@ public class ChoreService implements ChoreServicer {
try {
if (chore.getPeriod() <= 0) {
- LOG.info("The period is {} seconds, {} is disabled", chore.getPeriod(), chore.getName());
+ LOG.info("Chore {} is disabled because its period is not positive.", chore);
return false;
}
+ LOG.info("Chore {} is enabled.", chore);
chore.setChoreServicer(this);
ScheduledFuture<?> future =
scheduler.scheduleAtFixedRate(chore, chore.getInitialDelay(), chore.getPeriod(),