Re: Review Request 60604: AMBARI-21397. Remove previous upgrades to allow schema change

2017-07-03 Thread Attila Doroszlai
> On July 4, 2017, 1:21 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog252.java > > Lines 117-121 (patched) > > > > > > Can you please check to see what th

Re: Review Request 60586: Use the Correct Effective Stack ID for Commands During an Express Upgrade

2017-07-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60586/ --- (Updated July 4, 2017, 12:40 a.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 60586: Use the Correct Effective Stack ID for Commands During an Express Upgrade

2017-07-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60586/ --- (Updated July 4, 2017, 12:40 a.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 60604: AMBARI-21397. Remove previous upgrades to allow schema change

2017-07-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60604/#review179544 --- ambari-server/src/main/java/org/apache/ambari/server/upgrade/Upgr

Review Request 60604: AMBARI-21397. Remove previous upgrades to allow schema change

2017-07-03 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60604/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Sumit Mohan

Re: Review Request 60565: Force to use JDK 8 for Ambari Server maven build

2017-07-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60565/#review179516 --- Ship it! Ship It! - Sebastian Toader On July 3, 2017, 6:43 p

Re: Review Request 60565: Force to use JDK 8 for Ambari Server maven build

2017-07-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60565/ --- (Updated July 3, 2017, 4:43 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 60602: /usr/sbin/ambari-agent missing after Ambari upgrade

2017-07-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60602/#review179514 --- Ship it! Ship It! - Sumit Mohanty On July 3, 2017, 2:42 p.m.

Re: Review Request 60602: /usr/sbin/ambari-agent missing after Ambari upgrade

2017-07-03 Thread Victor Galgo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60602/#review179511 --- Ship it! Ship It! - Victor Galgo On July 3, 2017, 2:42 p.m.,

Re: Review Request 60595: AMBARI-21390. Change stack root references in config during cross-stack upgrade

2017-07-03 Thread Jonathan Hurley
> On July 3, 2017, 10:09 a.m., Jonathan Hurley wrote: > > I don't know why it's not showing Fix It, Then Ship It - but that's what I > > picked. Either way, I'll Ship It directly and you can correct the above > > comments. > > Attila Doroszlai wrote: > With the new design Review Board show

Review Request 60602: /usr/sbin/ambari-agent missing after Ambari upgrade

2017-07-03 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60602/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-21396 https:

Re: Review Request 60565: Force to use JDK 8 for Ambari Server maven build

2017-07-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60565/ --- (Updated July 3, 2017, 2:37 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 60595: AMBARI-21390. Change stack root references in config during cross-stack upgrade

2017-07-03 Thread Attila Doroszlai
> On July 3, 2017, 4:09 p.m., Jonathan Hurley wrote: > > I don't know why it's not showing Fix It, Then Ship It - but that's what I > > picked. Either way, I'll Ship It directly and you can correct the above > > comments. With the new design Review Board shows "Fix It" on top of "Ship It" (the

Re: Review Request 60595: AMBARI-21390. Change stack root references in config during cross-stack upgrade

2017-07-03 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60595/ --- (Updated July 3, 2017, 4:19 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 60595: AMBARI-21390. Change stack root references in config during cross-stack upgrade

2017-07-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60595/#review179505 --- Ship it! I don't know why it's not showing Fix It, Then Ship It

Re: Review Request 60595: AMBARI-21390. Change stack root references in config during cross-stack upgrade

2017-07-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60595/#review179502 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 60540: AMBARI-21339 logviewer started along with nimbus if supervisor is not running on the same machine

2017-07-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60540/#review179497 --- Ship it! Ship It! - Sebastian Toader On July 3, 2017, 11:53

Re: Review Request 60588: stackadvisor error during cluster creation, getOldValue is not defined

2017-07-03 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60588/#review179496 --- Ship it! Ship It! - Attila Doroszlai On July 3, 2017, 10:42

Re: Review Request 60540: AMBARI-21339 logviewer started along with nimbus if supervisor is not running on the same machine

2017-07-03 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60540/#review179493 --- Ship it! Ship It! - Attila Doroszlai On July 3, 2017, 11:53

Review Request 60595: AMBARI-21390. Change stack root references in config during cross-stack upgrade

2017-07-03 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60595/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Sumit Mohan

Re: Review Request 60588: stackadvisor error during cluster creation, getOldValue is not defined

2017-07-03 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60588/#review179492 --- Ship it! Ship It! - Sebastian Toader On July 3, 2017, 10:42

Re: Review Request 60540: AMBARI-21339 logviewer started along with nimbus if supervisor is not running on the same machine

2017-07-03 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60540/ --- (Updated July 3, 2017, 9:53 a.m.) Review request for Ambari, Attila Doroszlai,

Review Request 60590: Create a topic to send hostLevelParams

2017-07-03 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60590/ --- Review request for Ambari, Alejandro Fernandez, Myroslav Papirkovskyy, and Sid W

Re: Review Request 60588: stackadvisor error during cluster creation, getOldValue is not defined

2017-07-03 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60588/#review179485 --- Ship it! Ship It! - Laszlo Puskas On July 3, 2017, 8:42 a.m.

Review Request 60588: stackadvisor error during cluster creation, getOldValue is not defined

2017-07-03 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60588/ --- Review request for Ambari, Laszlo Puskas, Sebastian Toader, and Vitalyi Brodetsk

Re: Review Request 60463: Apply KerberosInterpreter related changes in Ambari

2017-07-03 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60463/ --- (Updated July 3, 2017, 12:52 p.m.) Review request for Ambari, Attila Doroszlai,