> On Січ. 31, 2017, 10:19 до полудня, Dmitro Lisnichenko wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java, > > line 759 > > <https://reviews.apache.org/r/55989/diff/4/?file=1619522#file1619522line759> > > > > is not this issue critical - maybe rethrow exception?
There was no exception before for a long time and i'm not sure it's good idea to add it. - Vitalyi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55989/#review163644 ----------------------------------------------------------- On Січ. 30, 2017, 11:12 після полудня, Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55989/ > ----------------------------------------------------------- > > (Updated Січ. 30, 2017, 11:12 після полудня) > > > 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 > >
