Browse Source

MAPREDUCE-6973. Fix comments on creating _SUCCESS file.

This closes #280

Signed-off-by: Akira Ajisaka <aajisaka@apache.org>
Mehul Garnara 7 years ago
parent
commit
c0a0c353e8

+ 2 - 1
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/lib/output/FileOutputCommitter.java

@@ -409,7 +409,7 @@ public class FileOutputCommitter extends PathOutputCommitter {
         LOG.info("Skip cleanup the _temporary folders under job's output " +
         LOG.info("Skip cleanup the _temporary folders under job's output " +
             "directory in commitJob.");
             "directory in commitJob.");
       } else {
       } else {
-        // delete the _temporary folder and create a _done file in the o/p
+        // delete the _temporary folder and create a _SUCCESS file in the o/p
         // folder
         // folder
         try {
         try {
           cleanupJob(context);
           cleanupJob(context);
@@ -426,6 +426,7 @@ public class FileOutputCommitter extends PathOutputCommitter {
       }
       }
       // True if the job requires output.dir marked on successful job.
       // True if the job requires output.dir marked on successful job.
       // Note that by default it is set to true.
       // Note that by default it is set to true.
+      // Create a _SUCCESS file in the o/p folder.
       if (context.getConfiguration().getBoolean(
       if (context.getConfiguration().getBoolean(
           SUCCESSFUL_JOB_OUTPUT_DIR_MARKER, true)) {
           SUCCESSFUL_JOB_OUTPUT_DIR_MARKER, true)) {
         Path markerPath = new Path(outputPath, SUCCEEDED_FILE_NAME);
         Path markerPath = new Path(outputPath, SUCCEEDED_FILE_NAME);