Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-15 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/#review185487 --- Ship it! Ship It! - Jonathan Hurley On Sept. 15, 2017, 9:35

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-15 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/ --- (Updated Sept. 15, 2017, 4:35 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Jonathan Hurley
> On Sept. 12, 2017, 11:59 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java > > Lines 415-449 (patched) > > > > > > Does it make sense

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Dmitro Lisnichenko
> On Sept. 12, 2017, 6:59 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java > > Lines 415-449 (patched) > > > > > > Does it make sense

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/#review185214 --- Ship it! Ship It! - Nate Cole On Sept. 12, 2017, 10:28

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/#review185213 ---

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/#review185203 ---