Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/#review183779 --- Ship it! Ship It! - Jonathan Hurley On Aug. 24, 2017, 2:46

Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-24 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/ --- (Updated Aug. 24, 2017, 6:46 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-24 Thread Sid Wagle
> On Aug. 24, 2017, 1:30 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java > > Lines 1428-1431 (original), 1444-1452 (patched) > > > > > > Is this

Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/#review183736 --- Fix it, then Ship it!

Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-23 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/ --- (Updated Aug. 23, 2017, 5:50 p.m.) Review request for Ambari, Myroslav

Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-23 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/ --- (Updated Aug. 23, 2017, 5:10 p.m.) Review request for Ambari, Myroslav

Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-23 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/ --- (Updated Aug. 23, 2017, 5:08 p.m.) Review request for Ambari, Myroslav

Re: Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-23 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/#review183581 ---

Review Request 61833: Ambari server logs consistency check warning after service delete with ConfigGroup present

2017-08-22 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/ --- Review request for Ambari, Myroslav Papirkovskyy, Sumit Mohanty, and Sebastian