----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33986/#review83470 -----------------------------------------------------------
Ship it! Ship It! - Sid Wagle On May 12, 2015, 8:30 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33986/ > ----------------------------------------------------------- > > (Updated May 12, 2015, 8:30 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 > > ambari-server/src/test/java/org/apache/ambari/server/controller/metrics/timeline/AMSPropertyProviderTest.java > 1573708 > > Diff: https://reviews.apache.org/r/33986/diff/ > > > Testing > ------- > > Manual tests passed. > Unit tests passed > > > File Attachments > ---------------- > > Can't update diff. Attached updated patch with fix for > AMSPropoertyProviderTest > > https://reviews.apache.org/media/uploaded/files/2015/05/12/1a0f6fec-df59-4408-91fa-03678d54f687__AMBARI-11030_2.patch > > > Thanks, > > Dmytro Sen > >
