[ https://issues.apache.org/jira/browse/HDFS-11147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15672048#comment-15672048 ]
Hudson commented on HDFS-11147: ------------------------------- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10850 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/10850/]) HDFS-11147. Remove confusing log output in (arp: rev 04a024b683cdaac8723fee5d606bc6da939e2fb9) * (edit) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/FsDatasetImpl.java > 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 > Fix For: 3.0.0-alpha2 > > Attachments: HDFS-11147.001.patch > > > 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-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org