Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HDFS-16108. Fix incorrect log placeholders used in JournalNodeSyncer (#3169)

This commit is contained in:
Viraj Jasani 2021-07-05 06:53:24 +05:30 committed by GitHub
parent b87bac13e4
commit da575a53cf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/qjournal/server/JournalNodeSyncer.java
View File

@ -175,7 +175,7 @@ public class JournalNodeSyncer {
} }
if (!createEditsSyncDir()) { if (!createEditsSyncDir()) {
LOG.error("Failed to create directory for downloading log " + LOG.error("Failed to create directory for downloading log " +
"segments: %s. Stopping Journal Node Sync.", "segments: {}. Stopping Journal Node Sync.",
journal.getStorage().getEditsSyncDir()); journal.getStorage().getEditsSyncDir());
return; return;
} }
@ -467,7 +467,7 @@ public class JournalNodeSyncer {
moveSuccess = journal.moveTmpSegmentToCurrent(tmpEditsFile, moveSuccess = journal.moveTmpSegmentToCurrent(tmpEditsFile,
finalEditsFile, log.getEndTxId()); finalEditsFile, log.getEndTxId());
} catch (IOException e) { } catch (IOException e) {
LOG.info("Could not move %s to current directory.", tmpEditsFile); LOG.info("Could not move {} to current directory.", tmpEditsFile);
} finally { } finally {
if (tmpEditsFile.exists() && !tmpEditsFile.delete()) { if (tmpEditsFile.exists() && !tmpEditsFile.delete()) {
LOG.warn("Deleting " + tmpEditsFile + " has failed"); LOG.warn("Deleting " + tmpEditsFile + " has failed");