bshashikant commented on a change in pull request #2290:
URL: https://github.com/apache/ozone/pull/2290#discussion_r648853731
##########
File path:
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/volume/StorageVolumeChecker.java
##########
@@ -109,11 +120,6 @@ public HddsVolumeChecker(ConfigurationSource conf, Timer
timer)
this.timer = timer;
DatanodeConfiguration dnConf = conf.getObject(DatanodeConfiguration.class);
- /**
- * Maximum number of volume failures that can be tolerated without
- * declaring a fatal error.
- */
- int maxVolumeFailuresTolerated = dnConf.getFailedVolumesTolerated();
minDiskCheckGapMs = conf.getTimeDuration(
DFSConfigKeysLegacy.DFS_DATANODE_DISK_CHECK_MIN_GAP_KEY,
Review comment:
I think, its better to avoid any legacy configs in the code. can be done
later as well.
##########
File path:
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/DatanodeLayout.java
##########
@@ -96,11 +97,14 @@ public static void main(String[] args) {
OzoneContainer.buildContainerSet(volumeSet, containerSet, conf);
volumeSet.shutdown();
+ List<HddsVolume> failedVolumes = StorageVolumeUtil.getHddsVolumesList(
+ volumeSet.getFailedVolumesList());
+
if (verify) {
- for (HddsVolume vol : volumeSet.getFailedVolumesList()) {
+ for (HddsVolume vol : failedVolumes) {
System.out.println("Failed Volume:" + vol.getHddsRootDir());
}
}
- return volumeSet.getFailedVolumesList();
+ return failedVolumes;
Review comment:
I think, its better to log if there is any failed metadata volume here
as well.
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
