It should be there since YARN-4878. But it doesn't.
This message was automatically generated.
Thanks, Yufei Gu. Two minor nits:
MetricsSource queueSource= TestQueueMetrics.queueSource(ms, queueName);
Thanks Daniel Templeton for the review. I uploaded new patch for your comments.
Thanks, Yufei Gu. +1 pending Jenkins.
Uploaded the new patch for the error. Seems like when you hide the static method, we cannot narrow the method privilege.
Looks like your new forQueue() should be synchronized and needs some javadoc.
Thanks Daniel Templeton. Uploaded patch 004 for your comments.
+1 on the latest patch.
Looks like you need to rebase after YARN-5890.
Uploaded patch 005 for the rebase.
Uploaded patch 006 to solve the style issue. The failed test is unrelated.
Thanks for the patch, Yufei Gu! I committed patch 5 because the style issue was preexisting, and you posting patch 6 while I was in the process of committing. Committed to branch-2 and trunk.
Thanks Daniel Templeton for the review and commit!
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10932 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10932/)
YARN-5929. Missing scheduling policy in the FS queue metric. (templedf: rev 5bd18c49bd5075fa20d24363dceea7828e3fa266)
This patch goes to branch-2 only instead of 2.8, set 2.9 as fix version.