It should be there since YARN-4878. But it doesn't.
Expose scheduling policy for each queue in FairScheduler Web UI
This patch goes to branch-2 only instead of 2.8, set 2.9 as fix version.
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)
Thanks Daniel Templeton for the review and commit!
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.
Uploaded patch 006 to solve the style issue. The failed test is unrelated.
This message was automatically generated.
Uploaded patch 005 for the rebase.
Looks like you need to rebase after YARN-5890.
+1 on the latest patch.
Thanks Daniel Templeton. Uploaded patch 004 for your comments.
Looks like your new forQueue() should be synchronized and needs some javadoc.
Uploaded the new patch for the error. Seems like when you hide the static method, we cannot narrow the method privilege.
Thanks, Yufei Gu. +1 pending Jenkins.
Thanks Daniel Templeton for the review. I uploaded new patch for your comments.
Thanks, Yufei Gu. Two minor nits:
MetricsSource queueSource= TestQueueMetrics.queueSource(ms, queueName);