ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sid Wagle" <swa...@hortonworks.com>
Subject Re: Review Request 33923: Error loading deferred resources when making metric API requests with null padding
Date Thu, 07 May 2015 00:48:31 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33923/
-----------------------------------------------------------

(Updated May 7, 2015, 12:48 a.m.)


Review request for Ambari, Alejandro Fernandez, Mahadev Konar, Sumit Mohanty, and Tom Beerbower.


Bugs: AMBARI-10987
    https://issues.apache.org/jira/browse/AMBARI-10987


Repository: ambari


Description
-------

If I add &params/padding=nulls to the metrics API request

/api/v1/clusters/c1/hosts/c6401.ambari.apache.org/host_components/NAMENODE?fields=metrics/cpu/cpu_system[1430833471,1430837071,15],metrics/cpu/cpu_user[1430833471,1430837071,15],metrics/cpu/cpu_nice[1430833471,1430837071,15],metrics/cpu/cpu_idle[1430833471,1430837071,15],metrics/cpu/cpu_wio[1430833471,1430837071,15],metrics/memory/mem_total[1430833471,1430837071,15],metrics/memory/mem_free[1430833471,1430837071,15],metrics/memory/mem_cached[1430833471,1430837071,15]&params/padding=nulls

fails:

{
  "status" : 500,
  "message" : "org.apache.ambari.server.controller.spi.SystemException: Error loading deferred
resources"
}


Diffs (updated)
-----

  ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/PhoenixHBaseAccessor.java
e27d9a9 
  ambari-server/src/main/java/org/apache/ambari/server/api/query/render/MetricsPaddingRenderer.java
PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/api/resources/BaseResourceDefinition.java
e63f7eb 
  ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsDownsamplingMethodFactory.java
a53e26d 
  ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsPropertyProvider.java
8360f5e 
  ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsPropertyProviderProxy.java
24f1851 
  ambari-server/src/test/java/org/apache/ambari/server/api/query/render/MetricsPaddingRendererTest.java
PRE-CREATION 

Diff: https://reviews.apache.org/r/33923/diff/


Testing
-------

Manually verified.
Related unit tests: ClusterControllerImplTest, All Renderer tests pass.


Thanks,

Sid Wagle


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message