-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57344/
-----------------------------------------------------------

(Updated March 9, 2017, 12:22 p.m.)


Review request for Ambari, Attila Doroszlai, Andrew Onischuk, Dmytro Sen, 
Robert Levas, and Sebastian Toader.


Bugs: AMBARI-20323
    https://issues.apache.org/jira/browse/AMBARI-20323


Repository: ambari


Description
-------

All status command execution logic moved under one file.

* added protection for hardkilling(now process tries to die gracefully)
* added protection for queue internal logic blocking(underlying pipes hard 
closed)
* added lock around queues that will prevent from using queues before new one 
will be created in case of recreating executor process


Diffs (updated)
-----

  ambari-agent/src/main/python/ambari_agent/ActionQueue.py 5300b52 
  ambari-agent/src/main/python/ambari_agent/Controller.py 61a74e6 
  ambari-agent/src/main/python/ambari_agent/StatusCommandsExecutor.py 2f15770 
  ambari-agent/src/main/python/ambari_agent/main.py 3f333c4 
  ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 9fefefb 
  ambari-agent/src/test/python/ambari_agent/TestController.py 663e215 
  ambari-agent/src/test/python/ambari_agent/TestMain.py 97c448b 


Diff: https://reviews.apache.org/r/57344/diff/5/

Changes: https://reviews.apache.org/r/57344/diff/4-5/


Testing
-------

mvn clean test, manual testing


Thanks,

Eugene Chekanskiy

Reply via email to