abstractdog commented on code in PR #339:
URL: https://github.com/apache/tez/pull/339#discussion_r1599573887


##########
tez-mapreduce/src/main/java/org/apache/tez/mapreduce/hadoop/MRJobConfig.java:
##########
@@ -131,6 +131,11 @@ public interface MRJobConfig {
 
   public static final String CACHE_ARCHIVES_VISIBILITIES = 
"mapreduce.job.cache.archives.visibilities";
 
+  /**
+   * Used by Hadoop's MagicS3Guard and Staging committers to set a job-wide 
UUID
+   */
+  public static final String FS_S3A_COMMITTER_UUID = "fs.s3a.committer.uuid";

Review Comment:
   I don't think this option should be "S3" specific, even if this is 
implemented due to S3
   the new option's name should tell this UUID is job-scoped



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to