Skip to content

Commit

Permalink
fix:the HdfsStorage can not delete checkpoint file apache#5046 (apach…
Browse files Browse the repository at this point in the history
  • Loading branch information
wu-a-ge authored and liunaijie committed Jul 13, 2023
1 parent 9df49f2 commit 9c8a8c8
Showing 1 changed file with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -283,7 +283,9 @@ public synchronized void deleteCheckpoint(String jobId, String pipelineId, Strin
if (pipelineId.equals(getPipelineIdByFileName(fileName))
&& checkpointId.equals(getCheckpointIdByFileName(fileName))) {
try {
fs.delete(new Path(fileName), false);
fs.delete(
new Path(path + DEFAULT_CHECKPOINT_FILE_PATH_SPLIT + fileName),
false);
} catch (Exception e) {
log.error(
"Failed to delete checkpoint {} for job {}, pipeline {}",
Expand Down Expand Up @@ -311,7 +313,9 @@ public void deleteCheckpoint(String jobId, String pipelineId, List<String> check
if (pipelineId.equals(getPipelineIdByFileName(fileName))
&& checkpointIdList.contains(checkpointIdByFileName)) {
try {
fs.delete(new Path(fileName), false);
fs.delete(
new Path(path + DEFAULT_CHECKPOINT_FILE_PATH_SPLIT + fileName),
false);
} catch (Exception e) {
log.error(
"Failed to delete checkpoint {} for job {}, pipeline {}",
Expand Down

0 comments on commit 9c8a8c8

Please sign in to comment.