Review Request 62584: Remove Database auto configuration from Infra Manager

2017-09-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62584/ --- Review request for Ambari, Attila Doroszlai, Laszlo Puskas, Miklos Gergely,

Re: Review Request 62571: Fail to restart Ranger Admin during HDP downgrade.

2017-09-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62571/#review186317 --- Ship it! Ship It! - Nate Cole On Sept. 26, 2017, 11:58

Review Request 62586: Update unit tests to test recursive variable replacements using VariableReplacementHelper

2017-09-26 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62586/ --- Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene Chekanskiy,

Re: Review Request 62586: Update unit tests to test recursive variable replacements using VariableReplacementHelper

2017-09-26 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62586/ --- (Updated Sept. 26, 2017, 4:12 p.m.) Review request for Ambari, Attila Magyar,

Re: Review Request 62584: Remove Database auto configuration from Infra Manager

2017-09-26 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62584/#review186307 --- Ship it! Ship It! - Attila Doroszlai On Sept. 26, 2017,

Re: Review Request 61886: AMBARI-21803. Implement STOMP endpoint for alert definitions

2017-09-26 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61886/#review186309 --- Ship it! Ship It! - Sid Wagle On Sept. 25, 2017, 2:19 p.m.,

Re: Review Request 62586: Update unit tests to test recursive variable replacements using VariableReplacementHelper

2017-09-26 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62586/#review186322 --- Ship it! Ship It! - Robert Nettleton On Sept. 26, 2017,

Re: Review Request 62586: Update unit tests to test recursive variable replacements using VariableReplacementHelper

2017-09-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62586/#review186340 --- Ship it! Ship It! - Jonathan Hurley On Sept. 26, 2017, 4:12

Re: Review Request 62584: Remove Database auto configuration from Infra Manager

2017-09-26 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62584/#review186321 --- Ship it! Ship It! - Robert Nettleton On Sept. 26, 2017,

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-26 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/#review186348 --- Hi Vitaly, Kindly refer https://reviews.apache.org/r/56727/ to

Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-26 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/ --- Review request for Ambari, Jayush Luniya and Madhuvanthi Radhakrishnan. Bugs:

Re: Review Request 62571: Fail to restart Ranger Admin during HDP downgrade.

2017-09-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62571/ --- (Updated Sept. 26, 2017, 6:58 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 62576: Solr Data Manager script should provide non-destructive archive download option

2017-09-26 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62576/#review186266 --- Ship it! Ship It! - Robert Nettleton On Sept. 26, 2017,

Re: Review Request 62413: Ambari upgrade don't delete stale constraints

2017-09-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62413/ --- (Updated Sept. 26, 2017, 6:55 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 62571: Fail to restart Ranger Admin during HDP downgrade.

2017-09-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62571/#review186275 --- Ship it! Ship It! - Jonathan Hurley On Sept. 26, 2017,

Re: Review Request 62578: Stack selection page does not load the HDP stacks (Redux)

2017-09-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62578/#review186287 --- Ship it! Ship It! - Dmitro Lisnichenko On Sept. 26, 2017,

Review Request 62578: Stack selection page does not load the HDP stacks (Redux)

2017-09-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62578/ --- Review request for Ambari, Dmitro Lisnichenko and Jonathan Hurley. Bugs:

Re: Review Request 62578: Stack selection page does not load the HDP stacks (Redux)

2017-09-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62578/#review186281 --- Ship it! Ship It! - Jonathan Hurley On Sept. 26, 2017,

Re: Review Request 62562: Solr Data Manager script should use gzip compression type

2017-09-26 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62562/#review186261 --- Ship it! Ship It! - Robert Nettleton On Sept. 26, 2017,

Re: Review Request 62571: Fail to restart Ranger Admin during HDP downgrade.

2017-09-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62571/#review186249 --- Ship it! Ship It! - Nate Cole On Sept. 26, 2017, 9:18 a.m.,

Re: Review Request 62570: Use symlinks for Infra components

2017-09-26 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62570/#review186262 --- Ship it! Ship It! - Robert Nettleton On Sept. 26, 2017,

Re: Review Request 62571: Fail to restart Ranger Admin during HDP downgrade.

2017-09-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62571/#review186256 ---

Re: Review Request 62413: (Preview) Ambari upgrade don't delete stale constraints

2017-09-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62413/#review186259 --- Ship it! Ship It! - Jonathan Hurley On Sept. 26, 2017, 9:24

Review Request 62576: Solr Data Manager script should provide non-destructive archive download option

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

Review Request 62570: Use symlinks for Infra components

2017-09-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62570/ --- Review request for Ambari, Attila Doroszlai, Andrew Onischuk, Miklos Gergely,

Re: Review Request 62570: Use symlinks for Infra components

2017-09-26 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62570/#review186248 --- Ship it! Ship It! - Miklos Gergely On Sept. 26, 2017, 1:09

Review Request 62571: Fail to restart Ranger Admin during HDP downgrade.

2017-09-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62571/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-22060

Re: Review Request 62413: (Preview) Ambari upgrade don't delete stale constraints

2017-09-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62413/ --- (Updated Sept. 26, 2017, 4:24 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 61910: Hive query failing randomly with Null point exception status 500

2017-09-26 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61910/#review186224 --- Ship it! Ship It! - Gaurav Nagar On Aug. 25, 2017, 7:16

Re: Review Request 62551: Make Log Search IT work with the new UI

2017-09-26 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62551/#review186219 --- Ship it! Ship It! - Miklos Gergely On Sept. 25, 2017, 6:56

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

2017-09-26 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62546/ --- (Updated Sept. 26, 2017, 2:04 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 61910: Hive query failing randomly with Null point exception status 500

2017-09-26 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61910/#review186225 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Aug. 25, 2017, 7:16

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

2017-09-26 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62546/#review186216 --- Ship it! Ship It! - Gaurav Nagar On Sept. 25, 2017, 3:02

Re: Review Request 62547: Zeppelin's version is wrong in alerts.json

2017-09-26 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62547/#review186214 --- Ship it! Ship It! - Gaurav Nagar On Sept. 25, 2017, 3:32

Re: Review Request 62563: Reduce alert reports performance overhead

2017-09-26 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62563/ --- (Updated Sept. 26, 2017, 10:54 a.m.) Review request for Ambari, Attila

Re: Review Request 62563: Reduce alert reports performance overhead

2017-09-26 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62563/ --- (Updated Sept. 26, 2017, 10:58 a.m.) Review request for Ambari, Attila

Re: Review Request 62563: Reduce alert reports performance overhead

2017-09-26 Thread Andrew Onischuk
> On Sept. 26, 2017, 10:14 a.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertReceivedListener.java > > Line 170 (original), 174 (patched) > > > > > >

Review Request 62563: Reduce alert reports performance overhead

2017-09-26 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62563/ --- Review request for Ambari, Attila Doroszlai, Myroslav Papirkovskyy, and Sid

Review Request 62562: Solr Data Manager script should use gzip compression type

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

Re: Review Request 62563: Reduce alert reports performance overhead

2017-09-26 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62563/#review186227 --- Fix it, then Ship it! Server-side change for this enhancement

Re: Review Request 62562: Solr Data Manager script should use gzip compression type

2017-09-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62562/#review186228 --- Ship it! Ship It! - Oliver Szabo On Sept. 26, 2017, 10:17

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

2017-09-26 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62546/ --- (Updated Sept. 26, 2017, 4:10 p.m.) Review request for Ambari, DIPAYAN

Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62612/ --- Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit

Re: Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62612/ --- (Updated Sept. 27, 2017, 11:13 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62612/#review186376 --- Ship it! Ship It! - Nitiraj Rathore On Sept. 27, 2017, 5:43

Re: Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62612/#review186377 --- Ship it! Ship It! - Gaurav Nagar On Sept. 27, 2017, 5:43