[ https://issues.apache.org/jira/browse/HDFS-17607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17885291#comment-17885291 ]
ASF GitHub Bot commented on HDFS-17607: --------------------------------------- Hexiaoqiao commented on code in PR #7012: URL: https://github.com/apache/hadoop/pull/7012#discussion_r1778337276 ########## 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: what about `datanode.getDNConf()` here? > Reduce the number of times conf is loaded when DataNode startUp > --------------------------------------------------------------- > > Key: HDFS-17607 > URL: https://issues.apache.org/jira/browse/HDFS-17607 > Project: Hadoop HDFS > Issue Type: Improvement > Reporter: lei w > Priority: Major > Labels: pull-request-available > > The value of the conf parameter in the current access method > StorageLocation#makeBlockPoolDir is null, which leads to the problem of > loading conf multiple times when DataNode startUp -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org