[hbase] branch branch-2 updated: HBASE-23283 Provide clear and consistent logging about the period of enabled chores

[bu...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

busbey pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new d22e6f5  HBASE-23283 Provide clear and consistent logging about the period of enabled chores
d22e6f5 is described below

commit d22e6f55d82af7a96790aec13e9dda24e2538a79
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(),