[helix] branch master updated: Reduce verbosity of some of the logs (#2584)

[xyuanlu]

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

xyuanlu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/helix.git


The following commit(s) were added to refs/heads/master by this push:
     new 2d60c5867 Reduce verbosity of some of the logs (#2584)
2d60c5867 is described below

commit 2d60c58676a1206f767798a06cf89cdb229674f5
Author: Komal Desai <98000016+desaiko...@users.noreply.github.com>
AuthorDate: Tue Aug 8 09:31:14 2023 -0700

    Reduce verbosity of some of the logs (#2584)
---
 .../helix/controller/rebalancer/waged/WagedInstanceCapacity.java      | 4 ++--
 .../apache/helix/controller/stages/TopStateHandoffReportStage.java    | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/WagedInstanceCapacity.java
 
b/helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/WagedInstanceCapacity.java
index d9bdaf879..29ecf451d 100644
--- 
a/helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/WagedInstanceCapacity.java
+++ 
b/helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/WagedInstanceCapacity.java
@@ -106,7 +106,7 @@ public class WagedInstanceCapacity implements 
InstanceCapacityDataProvider {
           // Get Partition Weight
           Map<String, Integer> partCapacity = 
weightProvider.getPartitionWeights(resName, partitionName);
           if (partCapacity == null || partCapacity.isEmpty()) {
-            LOG.info("Partition: " + partitionName + " in resource: " + resName
+            LOG.debug("Partition: " + partitionName + " in resource: " + 
resName
                 + " has no weight specified. Skipping it.");
             continue;
           }
@@ -184,7 +184,7 @@ public class WagedInstanceCapacity implements 
InstanceCapacityDataProvider {
       String partitionName, Map<String, Integer> partitionCapacity) {
 
     if (hasPartitionChargedForCapacity(instance, resName, partitionName)) {
-      LOG.info("Instance: " + instance + " for resource: " + resName
+      LOG.debug("Instance: " + instance + " for resource: " + resName
           + " for partition: " + partitionName + " already charged for 
capacity.");
       return true;
     }
diff --git 
a/helix-core/src/main/java/org/apache/helix/controller/stages/TopStateHandoffReportStage.java
 
b/helix-core/src/main/java/org/apache/helix/controller/stages/TopStateHandoffReportStage.java
index 3ba7773fd..6ca0771e2 100644
--- 
a/helix-core/src/main/java/org/apache/helix/controller/stages/TopStateHandoffReportStage.java
+++ 
b/helix-core/src/main/java/org/apache/helix/controller/stages/TopStateHandoffReportStage.java
@@ -282,7 +282,7 @@ public class TopStateHandoffReportStage extends 
AbstractBaseStage {
       // Top state handoff finished before end of last pipeline run, and 
instance contains
       // previous top state is no longer alive, so our best guess did not 
work, ignore the
       // data point for now.
-      LogUtil.logWarn(LOG, _eventId, String
+      LogUtil.logDebug(LOG, _eventId, String
           .format("Cannot confirm top state missing start time. %s:%s->%s. 
Likely it was very fast",
               partition.getPartitionName(), lastTopStateInstance, 
curTopStateInstance));
       return;