[HDFS-11147] Remove confusing log output in FsDatasetImpl#getInitialVolumeFailureInfos

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha2
• Component/s: datanode
• Hadoop Flags: Reviewed
• Assignee: Chen Liang
• Reporter: Chen Liang
• Created: 16/Nov/16 20:06
• Updated: 16/Nov/16 23:28
• Resolved: 16/Nov/16 22:49

Description

In FsDatasetImpl the method getInitialVolumeFailureInfos there is

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());
    }
}

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.