HDFS-4048. Use ERROR instead of INFO for volume failure logs. Contributed by Stephen Chu
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1407346 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
6b264008a9
commit
3d73261636
|
@ -111,6 +111,9 @@ Release 2.0.3-alpha - Unreleased
|
||||||
HDFS-4046. Rename ChecksumTypeProto enum NULL since it is illegal in
|
HDFS-4046. Rename ChecksumTypeProto enum NULL since it is illegal in
|
||||||
C/C++. (Binglin Chang via suresh)
|
C/C++. (Binglin Chang via suresh)
|
||||||
|
|
||||||
|
HDFS-4048. Use ERROR instead of INFO for volume failure logs.
|
||||||
|
(Stephen Chu via eli)
|
||||||
|
|
||||||
OPTIMIZATIONS
|
OPTIMIZATIONS
|
||||||
|
|
||||||
BUG FIXES
|
BUG FIXES
|
||||||
|
|
|
@ -431,7 +431,7 @@ public abstract class Storage extends StorageInfo {
|
||||||
if (!root.exists()) {
|
if (!root.exists()) {
|
||||||
// storage directory does not exist
|
// storage directory does not exist
|
||||||
if (startOpt != StartupOption.FORMAT) {
|
if (startOpt != StartupOption.FORMAT) {
|
||||||
LOG.info("Storage directory " + rootPath + " does not exist");
|
LOG.warn("Storage directory " + rootPath + " does not exist");
|
||||||
return StorageState.NON_EXISTENT;
|
return StorageState.NON_EXISTENT;
|
||||||
}
|
}
|
||||||
LOG.info(rootPath + " does not exist. Creating ...");
|
LOG.info(rootPath + " does not exist. Creating ...");
|
||||||
|
@ -440,15 +440,15 @@ public abstract class Storage extends StorageInfo {
|
||||||
}
|
}
|
||||||
// or is inaccessible
|
// or is inaccessible
|
||||||
if (!root.isDirectory()) {
|
if (!root.isDirectory()) {
|
||||||
LOG.info(rootPath + "is not a directory");
|
LOG.warn(rootPath + "is not a directory");
|
||||||
return StorageState.NON_EXISTENT;
|
return StorageState.NON_EXISTENT;
|
||||||
}
|
}
|
||||||
if (!root.canWrite()) {
|
if (!root.canWrite()) {
|
||||||
LOG.info("Cannot access storage directory " + rootPath);
|
LOG.warn("Cannot access storage directory " + rootPath);
|
||||||
return StorageState.NON_EXISTENT;
|
return StorageState.NON_EXISTENT;
|
||||||
}
|
}
|
||||||
} catch(SecurityException ex) {
|
} catch(SecurityException ex) {
|
||||||
LOG.info("Cannot access storage directory " + rootPath, ex);
|
LOG.warn("Cannot access storage directory " + rootPath, ex);
|
||||||
return StorageState.NON_EXISTENT;
|
return StorageState.NON_EXISTENT;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -137,7 +137,7 @@ class FsVolumeList {
|
||||||
if (removedVols != null && removedVols.size() > 0) {
|
if (removedVols != null && removedVols.size() > 0) {
|
||||||
// Replace volume list
|
// Replace volume list
|
||||||
volumes = Collections.unmodifiableList(volumeList);
|
volumes = Collections.unmodifiableList(volumeList);
|
||||||
FsDatasetImpl.LOG.info("Completed checkDirs. Removed " + removedVols.size()
|
FsDatasetImpl.LOG.warn("Completed checkDirs. Removed " + removedVols.size()
|
||||||
+ " volumes. Current volumes: " + this);
|
+ " volumes. Current volumes: " + this);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue