ThinkerLei commented on code in PR #7012:
URL: https://github.com/apache/hadoop/pull/7012#discussion_r1796360139


##########
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataStorage.java:
##########
@@ -335,7 +335,7 @@ public VolumeBuilder prepareVolume(DataNode datanode,
     VolumeBuilder builder =
         new VolumeBuilder(this, sd);
     for (NamespaceInfo nsInfo : nsInfos) {
-      location.makeBlockPoolDir(nsInfo.getBlockPoolID(), null);
+      location.makeBlockPoolDir(nsInfo.getBlockPoolID(), datanode.getConf());

Review Comment:
   @Hexiaoqiao Thank you for your response. The method cannot use getDnConf 
here because the type used inside the method is Configuration, not DNConf.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to