----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/#review163966 -----------------------------------------------------------
ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py (line 320) <https://reviews.apache.org/r/56225/#comment235476> Hi Attila. Checking if the key in dictionary before operation looks less hacky to me. What do you think? - Andrew Onischuk On Feb. 2, 2017, 1:12 p.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56225/ > ----------------------------------------------------------- > > (Updated Feb. 2, 2017, 1:12 p.m.) > > > Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Sandor > Magyari, and Sebastian Toader. > > > Bugs: AMBARI-19846 > https://issues.apache.org/jira/browse/AMBARI-19846 > > > Repository: ambari > > > Description > ------- > > Avoid recording the traceback for this `KeyError`, which is expected for > status commands. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py > 9baaf08d6cd1fe38db19a34907d44514bded228d > ambari-agent/src/test/python/ambari_agent/TestCustomServiceOrchestrator.py > 3985c5a21ebcaa5c526b772e8ad13d1bf0427d99 > > Diff: https://reviews.apache.org/r/56225/diff/ > > > Testing > ------- > > Manually tested on local cluster. > > Unit tests: > > ``` > $ mvn -pl ambari-agent clean test > ... > Ran 455 tests in 15.152s > > OK > ... > [INFO] BUILD SUCCESS > ``` > > > Thanks, > > Attila Doroszlai > >
