-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20889/
-----------------------------------------------------------
(Updated April 30, 2014, 11:51 a.m.)
Review request for Ambari and Vitalyi Brodetskyi.
Bugs: AMBARI-5633
https://issues.apache.org/jira/browse/AMBARI-5633
Repository: ambari
Description
-------
Now execution of status command takes ~100 milliseconds per command. Also added hostname caching
and noticed, that every 2 or 3 status commands heartbeat was generated. This can cause high
server load on clusters with large number of nodes. Now status command don't cause often heartbeats.
Diffs (updated)
-----
ambari-agent/src/main/python/ambari_agent/CommandStatusDict.py 15e7e2e
ambari-agent/src/main/python/ambari_agent/hostname.py 063351e
ambari-agent/src/test/python/ambari_agent/TestCommandStatusDict.py 8751479
ambari-agent/src/test/python/ambari_agent/TestHostname.py 7d447b5
Diff: https://reviews.apache.org/r/20889/diff/
Testing
-------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Main ....................................... SUCCESS [1.897s]
[INFO] Apache Ambari Project POM ......................... SUCCESS [0.306s]
[INFO] Ambari Web ........................................ SUCCESS [5.150s]
[INFO] Ambari Views ...................................... SUCCESS [1.445s]
[INFO] Ambari Server ..................................... SUCCESS [12:54.207s]
[INFO] Ambari Agent ...................................... SUCCESS [6.012s]
[INFO] Ambari Client ..................................... SUCCESS [0.268s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 13:10.048s
[INFO] Finished at: Wed Apr 30 14:08:28 FET 2014
[INFO] Final Memory: 25M/179M
[INFO] ------------------------------------------------------------------------
Thanks,
Dmitro Lisnichenko
|