----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51711/#review148543 -----------------------------------------------------------
Ship it! Ship It! - Sid Wagle On Sept. 11, 2016, 2:10 a.m., Aravindan Vijayan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51711/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2016, 2:10 a.m.) > > > Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-17458 > https://issues.apache.org/jira/browse/AMBARI-17458 > > > Repository: ambari > > > Description > ------- > > Support Multiple collectors in Ambari Server, managed through a separate > MetricsHAManager module. Round robin scheduling with 5mins time slice is used > for toggling between available collectors. Ambari event framework is used to > notify the MetricsHAManager about a down collector. > > This patch also fixes the bug in reporting AMS components' process status. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java > 9da6fd4 > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 1d82928 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java > b6cbed5 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostStatusHelper.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsCollectorHAClusterState.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsCollectorHAManager.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/AMSPropertyProvider.java > 9e81df4 > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/AMSReportPropertyProvider.java > 3688742 > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/MetricsRequestHelper.java > 1df0f6a > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/cache/TimelineMetricCache.java > b5fe05e > > ambari-server/src/main/java/org/apache/ambari/server/events/AmbariEvent.java > 912c441 > > ambari-server/src/main/java/org/apache/ambari/server/events/MetricsCollectorHostDownEvent.java > PRE-CREATION > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/metrics_collector.py > 2c7119b > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/metrics_grafana.py > 747a687 > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/metrics_monitor.py > d53c1fc > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/status.py > 14af3ad > > Diff: https://reviews.apache.org/r/51711/diff/ > > > Testing > ------- > > Manually tested. > Unit tests pending. > > > Thanks, > > Aravindan Vijayan > >
