----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55989/#review163647 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java <https://reviews.apache.org/r/55989/#comment235128> Please don't move imports unnecessarily. - Attila Doroszlai On Jan. 31, 2017, 12:12 a.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55989/ > ----------------------------------------------------------- > > (Updated Jan. 31, 2017, 12:12 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Myroslav Papirkovskyy, and > Sid Wagle. > > > Repository: ambari > > > Description > ------- > > I've deployed cluster with 3000 agents and started my little test script on > it. Script is doing only three actions in loop: 1) Stop All services 2) Start > All services 3) Update zk config. > After few days of work these actions need much more time to be executed. For > example before stop/start all actions took near 7-8 minutes. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java > c53a73d > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java > 93f4a8f > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 0fce72b > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java > d304bb3 > ambari-server/src/main/java/org/apache/ambari/server/state/Host.java > 04b2104 > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java > 0381e38 > > ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java > 3c7b040 > > ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java > 1bcffe4 > > Diff: https://reviews.apache.org/r/55989/diff/ > > > Testing > ------- > > > Thanks, > > Vitalyi Brodetskyi > >
