jojochuang commented on a change in pull request #1490:
URL: https://github.com/apache/hadoop-ozone/pull/1490#discussion_r503922135



##########
File path: 
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/utils/HddsVolumeUtil.java
##########
@@ -212,6 +212,9 @@ public static boolean checkVolume(HddsVolume hddsVolume, 
String scmId, String
     } else {
       // The hdds root dir should always have 2 files. One is Version file
       // and other is SCM directory.
+      logger.error("The hdds root dir {} should always have 2 files. " +
+              "One is Version file and other is SCM directory.",

Review comment:
       Let's add a more actionable message: "Please remove any other extra 
files from the directory so that DataNode startup can proceed."




----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org

Reply via email to