Skip to content

Commit d4f7e25

Browse files
committed
MAPREDUCE-6282. Reuse historyFileAbsolute.getFileSystem in CompletedJob#loadFullHistoryData for code optimization. (zxu via rkanter)
1 parent 75ead27 commit d4f7e25

File tree

2 files changed

+5
-3
lines changed
  • hadoop-mapreduce-project

2 files changed

+5
-3
lines changed

hadoop-mapreduce-project/CHANGES.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -274,6 +274,10 @@ Release 2.8.0 - UNRELEASED
274274
MAPREDUCE-5755. MapTask.MapOutputBuffer#compare/swap should have
275275
@Override annotation. (ozawa)
276276

277+
MAPREDUCE-6282. Reuse historyFileAbsolute.getFileSystem in
278+
CompletedJob#loadFullHistoryData for code optimization.
279+
(zxu via rkanter)
280+
277281
OPTIMIZATIONS
278282

279283
BUG FIXES

hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/main/java/org/apache/hadoop/mapreduce/v2/hs/CompletedJob.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -345,9 +345,7 @@ protected synchronized void loadFullHistoryData(boolean loadTasks,
345345
JobHistoryParser parser = null;
346346
try {
347347
final FileSystem fs = historyFileAbsolute.getFileSystem(conf);
348-
parser =
349-
new JobHistoryParser(historyFileAbsolute.getFileSystem(conf),
350-
historyFileAbsolute);
348+
parser = new JobHistoryParser(fs, historyFileAbsolute);
351349
final Path jobConfPath = new Path(historyFileAbsolute.getParent(),
352350
JobHistoryUtils.getIntermediateConfFileName(jobId));
353351
final Configuration conf = new Configuration();

0 commit comments

Comments
 (0)