It would be valuable to know the resource consumed by a container in the RM audit log.
I didn't do a careful review yet, but the patch looks reasonable. I don't see any obvious red flags.
+1 lgtm. I'll commit this tomorrow if there are no objections.
I see this is targeted to 3.0.0. Given the audit log is already key,value pairs and should be parsed as such, is there a reason this shouldn't go into 2.9.0 as well?
Thanks to Nathan Roberts for the contribution and to Daniel Templeton for additional review! I committed this to trunk and branch-2.
FAILURE: Integrated in Hadoop-trunk-Commit #9708 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9708/)
YARN-5003. Add container resource to RM audit log. Contributed by Nathan (jlowe: rev ed54f5f1ff7862f8216f77d5ea8f9ccea674ccd1)