Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137455 --- Ship it! Ship It! - DIPAYAN BHOWMICK On June 13, 2016, 9:41

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137354 --- Ship it! Ship It! - Alejandro Fernandez On June 13, 2016, 9:

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137353 --- ambari-server/src/main/java/org/apache/ambari/server/upgrade/Upgr

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Gaurav Nagar
> On June 13, 2016, 6:37 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > line 2440 > > > > > > Move these properties out of the for-loop

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137351 --- ambari-server/src/main/java/org/apache/ambari/server/upgrade/Upgr

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137264 --- Ship it! Ship It! - Nitiraj Rathore On June 13, 2016, 9:41 a