MAPREDUCE-5708. Duplicate String.format in

YarnOutputFiles.getSpillFileForWrite. Contributed by Konstantin Weitz.

hadoop-mapreduce-project/CHANGES.txt

@@ -422,6 +422,9 @@ Release 2.8.0 - UNRELEASED
     MAPREDUCE-6366. mapreduce.terasort.final.sync configuration in TeraSort
     doesn't work. (Takuya Fukudome via ozawa)
 
+    MAPREDUCE-5708. Duplicate String.format in YarnOutputFiles.getSpillFileForWrite.
+    (Konstantin Weitz via devaraj)
+
 Release 2.7.1 - UNRELEASED
 
   INCOMPATIBLE CHANGES

hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapred/YarnOutputFiles.java

@@ -157,8 +157,8 @@ public class YarnOutputFiles extends MapOutputFile {
   public Path getSpillFileForWrite(int spillNumber, long size)
       throws IOException {
     return lDirAlloc.getLocalPathForWrite(
-        String.format(String.format(SPILL_FILE_PATTERN,
-            conf.get(JobContext.TASK_ATTEMPT_ID), spillNumber)), size, conf);
+        String.format(SPILL_FILE_PATTERN,
+            conf.get(JobContext.TASK_ATTEMPT_ID), spillNumber), size, conf);
   }
 
   /**