----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45676/ -----------------------------------------------------------
(Updated 4月 19, 2016, 11:52 p.m.) Review request for Ambari, Andrew Onischuk, Jaimin Jetly, jun aoki, Jayush Luniya, 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 (updated) ----- 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