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

Review request for Ambari, jun aoki and Vitalyi Brodetskyi.


Repository: ambari


Description
-------

If yarn app state is not state or yarn app finalStatus is not succeeded, YARN 
service check should fail.
But in the YARN service_check.py, it doesn't fail because raise statement is in 
try block and there is only `pass` in except block.


Diffs
-----

  
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service_check.py
 e8d98ab 

Diff: https://reviews.apache.org/r/45676/diff/


Testing
-------

mvn clean test


Thanks,

Masahiro Tanaka

Reply via email to