ayushtkn commented on a change in pull request #1554:
URL: https://github.com/apache/ozone/pull/1554#discussion_r517413157
##########
File path:
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/IncrementalContainerReportHandler.java
##########
@@ -87,7 +89,7 @@ public void onMessage(final
IncrementalContainerReportFromDatanode report,
report.getDatanodeDetails(), ex);
} catch (IOException e) {
success = false;
- LOG.error("Exception while processing ICR for container {}",
+ LOG.warn("Exception while processing ICR for container {}",
Review comment:
I think rather than changing here, we should catch a separate
`ContainerReplicaNotFoundException ` and there reduce the log level,
suppressing for all IOE might not be a good choice.
@jojochuang does that make sense?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
