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