You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-dev@hadoop.apache.org by "Chen Liang (JIRA)" <ji...@apache.org> on 2016/11/16 20:06:58 UTC

[jira] [Created] (HDFS-11147) Remove confusing log output in FsDatasetImpl#getInitialVolumeFailureInfos

Chen Liang created HDFS-11147:
---------------------------------

             Summary: Remove confusing log output in FsDatasetImpl#getInitialVolumeFailureInfos
                 Key: HDFS-11147
                 URL: https://issues.apache.org/jira/browse/HDFS-11147
             Project: Hadoop HDFS
          Issue Type: Improvement
          Components: datanode
            Reporter: Chen Liang
            Assignee: Chen Liang
            Priority: Minor


In {{FsDatasetImpl}} the method {{getInitialVolumeFailureInfos}} there is 
{code}
for (StorageLocation sl: dataLocations) {
      LOG.info("Adding to failedLocationSet " + sl);
      failedLocationSet.add(sl);
    }
    for (Iterator<Storage.StorageDirectory> it = storage.dirIterator();
         it.hasNext(); ) {
      Storage.StorageDirectory sd = it.next();
      failedLocationSet.remove(sd.getStorageLocation());
      LOG.info("Removing from failedLocationSet " + sd.getStorageLocation());
    }
}
{code}
The log messages here can be confusing as it may prints "Adding to failedLocationSet" message, which looks like a failure is happening, but later on removed the storage location from failedLocationSet. We should just remove these two log messages.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-help@hadoop.apache.org