[GitHub] [ozone] siddhantsangwan commented on a diff in pull request #5211: HDDS-9204. Reduce log level of PipelinePlacementPolicy failed to create pipeline message

["siddhantsangwan (via GitHub)" <gi...@apache.org>]

siddhantsangwan commented on code in PR #5211:
URL: https://github.com/apache/ozone/pull/5211#discussion_r1305276180


##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelinePlacementPolicy.java:
##########
@@ -413,13 +413,11 @@ private boolean getAnchorAndNextNode(List<DatanodeDetails> healthyNodes,
         removePeers(anchor, healthyNodes);
         mutableExclude.add(anchor);
       } else {
-        LOG.warn("Unable to find healthy node for anchor(first) node.");
+        LOG.debug("Unable to find healthy node for anchor(first) node.");
         throw new SCMException("Unable to find anchor node.",

Review Comment:
   I wonder if we should include the entire message in the exception here as well, like in other places. Same comment for the change above this one.



-- 
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: issues-unsubscribe@ozone.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org