imbajin commented on a change in pull request #2265:
URL: https://github.com/apache/hadoop/pull/2265#discussion_r481554949
##########
File path:
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSInputStream.java
##########
@@ -181,6 +181,8 @@ private boolean isPeriodicRefreshEnabled() {
private byte[] oneByteBuf; // used for 'int read()'
protected void addToLocalDeadNodes(DatanodeInfo dnInfo) {
+ DFSClient.LOG.debug("Add {} to local dead nodes, previously was {}",
Review comment:
1. From the statistics of the production environment, the log output of
local `deadnode` information is too few, so that it is troublesome to find the
problem reason
2. The reason to use the `DEBUG` log level is to separate from the `INFO`
log level of the detector, and the impact is small.
3. Remove logs can be added at the same time(I think both are OK). It was
not added before considering that the local dead node removal method will
almost never be called, unless the detector is turned on and there is already
log output there.
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
