Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

Commit

Permalink
Fix #1608, Fix file mover OOM issue (#1610)
Browse files Browse the repository at this point in the history
  • Loading branch information
littlezhou authored Mar 2, 2018
1 parent 490df26 commit d317928
Showing 1 changed file with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,12 @@ public int executeMove(FileMovePlan plan, PrintStream resultOs, PrintStream logO
parseSchedulePlan(plan);

moveExecutor = Executors.newFixedThreadPool(maxConcurrentMoves);
return doMove(resultOs, logOs);
try {
return doMove(resultOs, logOs);
} finally {
moveExecutor.shutdown();
moveExecutor = null;
}
}

/**
Expand Down

0 comments on commit d317928

Please sign in to comment.