GeorgeJahad commented on code in PR #3980:
URL: https://github.com/apache/ozone/pull/3980#discussion_r1131539298
##########
hadoop-hdds/rocksdb-checkpoint-differ/src/main/java/org/apache/ozone/rocksdiff/RocksDBCheckpointDiffer.java:
##########
@@ -164,19 +164,19 @@ public RocksDBCheckpointDiffer(String metadataDir, String
sstBackupDir,
this.activeDBLocationStr = activeDBLocation.toString() + "/";
}
- private void setCompactionLogDir(String metadataDir,
+ private void setCompactionLogDir(String snapshotDiffDir,
String compactionLogDirName) {
- final File parentDir = new File(metadataDir);
+ final File parentDir = new File(snapshotDiffDir);
if (!parentDir.exists()) {
- if (!parentDir.mkdir()) {
+ if (!parentDir.mkdirs()) {
LOG.error("Error creating compaction log parent dir.");
return;
}
}
this.compactionLogDir =
- Paths.get(metadataDir, compactionLogDirName).toString();
+ Paths.get(parentDir.toString(), compactionLogDirName).toString();
Review Comment:
This change got superseded by changes in master
--
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]
