----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46751/ -----------------------------------------------------------
Review request for Ambari, Aravindan Vijayan, Dmytro Sen, Jonathan Hurley, and Sumit Mohanty. Bugs: AMBARI-16143 https://issues.apache.org/jira/browse/AMBARI-16143 Repository: ambari Description ------- In case of a SocketTimeoutException skip populating the resource with metrics until the next retry. Diffs ----- ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/AMSPropertyProvider.java 939ee7f ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/AMSReportPropertyProvider.java 8b60c50 ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/MetricsRequestHelper.java ca20e54 ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/cache/TimelineMetricCache.java 9e343e3 ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/cache/TimelineMetricCacheEntryFactory.java e8a2aef ambari-server/src/test/java/org/apache/ambari/server/controller/metrics/timeline/AMSPropertyProviderTest.java ce865cf Diff: https://reviews.apache.org/r/46751/diff/ Testing ------- Added a unit test. Manually verified the patch works on a cluster. Thanks, Sid Wagle