MAPREDUCE-6528. Memory leak for HistoryFileManager.getJobSummary(). Contributed by Junping Du

(cherry picked from commit 6344b6a769)
This commit is contained in:
Jason Lowe 2015-10-30 15:31:38 +00:00
parent 055e11c53e
commit a7d8895b60
2 changed files with 16 additions and 3 deletions

View File

@ -390,6 +390,9 @@ Release 2.7.2 - UNRELEASED
avoid FileNotFoundException causing HistoryFileInfo into MOVE_FAILED state.
(zhihai xu via devaraj)
MAPREDUCE-6528. Memory leak for HistoryFileManager.getJobSummary()
(Junping Du via jlowe)
Release 2.7.1 - 2015-07-06
INCOMPATIBLE CHANGES
@ -620,6 +623,9 @@ Release 2.6.3 - UNRELEASED
avoid FileNotFoundException causing HistoryFileInfo into MOVE_FAILED state.
(zhihai xu via devaraj)
MAPREDUCE-6528. Memory leak for HistoryFileManager.getJobSummary()
(Junping Du via jlowe)
Release 2.6.2 - 2015-10-21
INCOMPATIBLE CHANGES

View File

@ -966,9 +966,16 @@ public class HistoryFileManager extends AbstractService {
private String getJobSummary(FileContext fc, Path path) throws IOException {
Path qPath = fc.makeQualified(path);
FSDataInputStream in = fc.open(qPath);
String jobSummaryString = in.readUTF();
in.close();
FSDataInputStream in = null;
String jobSummaryString = null;
try {
in = fc.open(qPath);
jobSummaryString = in.readUTF();
} finally {
if (in != null) {
in.close();
}
}
return jobSummaryString;
}