----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33986/#review83032 -----------------------------------------------------------
Does this stop ambari server from requesting all queue metrics from the backend? What I observed (without this patch) is that besides the regex replace not working, multiple queue metrics were requested even though only /AppsCompleted was part of the API call. - Sid Wagle On May 8, 2015, 3:47 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33986/ > ----------------------------------------------------------- > > (Updated May 8, 2015, 3:47 p.m.) > > > Review request for Ambari, Jonathan Hurley, Myroslav Papirkovskyy, and Sid > Wagle. > > > Bugs: AMBARI-11030 > https://issues.apache.org/jira/browse/AMBARI-11030 > > > Repository: ambari > > > Description > ------- > > Queue-specific metrics cannot be retrieved as time-series data via Ambari API. > /api/v1/clusters/c1/services/YARN/components/RESOURCEMANAGER?fields=metrics/yarn/Queue/root/default/AppsRunning > returns the point-in-time metric properly. > However, when you query for it as time-series, in the following format the > API returns nothing: > /api/v1/clusters/c1/services/YARN/components/RESOURCEMANAGER?fields=metrics/yarn/Queue/root/default/AppsRunning[start,end,interval] > The time-series query works if you replace /root/default with /root. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BaseProvider.java > a95342a > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BaseProviderTest.java > 380142e > > Diff: https://reviews.apache.org/r/33986/diff/ > > > Testing > ------- > > Manual tests passed. > Unit tests execution is in progress > > > Thanks, > > Dmytro Sen > >
