avermeer commented on code in PR #3597:
URL: https://github.com/apache/storm/pull/3597#discussion_r1387937669


##########
storm-server/src/main/java/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java:
##########
@@ -194,7 +194,11 @@ private Set<String> 
refreshBlacklistedSupervisorIds(Cluster cluster, Topologies
                 new ArrayList<>(badSupervisorsToleranceSlidingWindow),
                 new ArrayList<>(sendAssignmentFailureCount),
                 cluster, topologies);
-        LOG.info("Supervisors {} are blacklisted.", blacklistedSupervisors);
+        if (blacklistedSupervisors.isEmpty()) {
+            LOG.debug("No Supervisors are blacklisted.");

Review Comment:
   No big deal, let's consider it for post-Storm 2.6.0 release, and thanks for 
this fix!



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@storm.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to