Re: Review Request 60645: Fix Unit Test Failures Caused by AMBARI-21391

2017-07-04 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60645/#review179594 --- Ship it! Ship It! - Sid Wagle On July 5, 2017, 3:45 a.m.,

Review Request 60645: Fix Unit Test Failures Caused by AMBARI-21391

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

Re: Review Request 60249: AMBARI-21234 Ambari rack awareness for Kafka

2017-07-04 Thread Ambud Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60249/ --- (Updated July 4, 2017, 6:53 p.m.) Review request for Ambari and Alejandro

Re: Review Request 60634: Create property descriptions for internal Log Feeder configs

2017-07-04 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60634/#review179588 --- Ship it! Ship It! - Oliver Szabo On July 4, 2017, 3:44

Review Request 60634: Create property descriptions for internal Log Feeder configs

2017-07-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60634/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs:

Re: Review Request 60637: Cleanup relevant Kerberos identities when a service is removed

2017-07-04 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60637/#review179583 --- Ship it! Ship It! - Sebastian Toader On July 4, 2017, 5:20

Re: Review Request 60637: Cleanup relevant Kerberos identities when a service is removed

2017-07-04 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60637/ --- (Updated July 4, 2017, 3:20 p.m.) Review request for Ambari, Balázs Bence

Re: Review Request 60637: Cleanup relevant Kerberos identities when a service is removed

2017-07-04 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60637/ --- (Updated July 4, 2017, 3:06 p.m.) Review request for Ambari, Balázs Bence

Review Request 60637: Cleanup relevant Kerberos identities when a service is removed

2017-07-04 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60637/ --- Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Robert Levas, and

Re: Review Request 60642: Support cancel commands, some changes to status/command reports, bugfixes

2017-07-04 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60642/#review179579 --- Ship it! Ship It! - Dmytro Sen On Июль 4, 2017, 2:33 п.п.,

Review Request 60642: Support cancel commands, some changes to status/command reports, bugfixes

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

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

2017-07-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60604/#review179573 --- Ship it! Ship It! - Jonathan Hurley On July 3, 2017, 2:05

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

2017-07-04 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60586/#review179572 --- Ship it! Ship It! - Attila Doroszlai On July 4, 2017, 6:40

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

2017-07-04 Thread Jonathan Hurley
> On July 4, 2017, 6:21 a.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java > > Line 369 (original), 371 (patched) > > > > > >

Re: Review Request 60249: AMBARI-21234 Ambari rack awareness for Kafka

2017-07-04 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60249/#review179569 --- ```true``` needs to be added to the ```metainfo.xml``` for Kafka

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

2017-07-04 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60565/#review179566 --- Ship it! Ship It! - Attila Magyar On July 4, 2017, 11:59

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

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

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

2017-07-04 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60586/#review179555 ---

Re: Review Request 60590: Create a topic to send hostLevelParams

2017-07-04 Thread Victor Galgo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60590/#review179552 --- Ship it! Ship It! - Victor Galgo On July 3, 2017, 9:36

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

2017-07-04 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