Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-25 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61856/ --- (Updated Сер. 25, 2017, 6:33 після полудня) Review request for Ambari,

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-25 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61856/#review183864 --- Fix it, then Ship it!

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-25 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61856/ --- (Updated Сер. 25, 2017, 3:34 після полудня) Review request for Ambari,

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-23 Thread Vitalyi Brodetskyi
> On Сер. 23, 2017, 6:59 після полудня, Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog260.java > > Lines 160 (patched) > > > > > > Why doesn't this check is service

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-23 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61856/ --- (Updated Сер. 23, 2017, 9:14 після полудня) Review request for Ambari,

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-23 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61856/#review183641 ---