Re: Review Request 51961: Prerequisite checks should only check services that are getting upgraded

2016-09-19 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51961/#review149442 --- Ship it!

Re: Review Request 51961: Prerequisite checks should only check services that are getting upgraded

2016-09-16 Thread Nate Cole
> On Sept. 16, 2016, 1:54 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/AbstractCheckDescriptor.java, > > line 161 > > > > > > %s but no param? Ha! Good catch; will

Re: Review Request 51961: Prerequisite checks should only check services that are getting upgraded

2016-09-16 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51961/#review149223 --- Fix it, then Ship it!

Re: Review Request 51961: Prerequisite checks should only check services that are getting upgraded

2016-09-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51961/#review149217 --- Ship it! Ship It! - Alejandro Fernandez On Sept. 16, 2016,

Review Request 51961: Prerequisite checks should only check services that are getting upgraded

2016-09-16 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51961/ --- Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and