Re: Review Request 51761: AMBARI-18050 - Upgrade pre-req check code needs to be decoupled from CheckDescription class

2016-09-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51761/#review148496 --- Perhaps we should have a check interface instead of requiring

Re: Review Request 51761: AMBARI-18050 - Upgrade pre-req check code needs to be decoupled from CheckDescription class

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

Re: Review Request 51761: AMBARI-18050 - Upgrade pre-req check code needs to be decoupled from CheckDescription class

2016-09-09 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51761/#review148347 --- Ship it! Ship It! - Di Li On Sept. 9, 2016, 4:59 p.m., Tim

Review Request 51761: AMBARI-18050 - Upgrade pre-req check code needs to be decoupled from CheckDescription class

2016-09-09 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51761/ --- Review request for Ambari, Alejandro Fernandez, Di Li, Jayush Luniya, and Nate