Re: Review Request 51203: Doc updates about ldap sync related properties

2016-08-18 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51203/#review146170 --- Ship it! Ship It! - Sebastian Toader On Aug. 18, 2016, 2:40

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Sumit Mohanty
> On Aug. 18, 2016, 8:54 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py, > > line 71 > > > > > > Can we call kinit if Kerberos is no

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146163 --- Ship it! Ship It! - Sumit Mohanty On Aug. 18, 2016, 7:42 p.m

Re: Review Request 51203: Doc updates about ldap sync related properties

2016-08-18 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51203/#review146143 --- Ship it! Ship It! - Jonathan Hurley On Aug. 18, 2016, 8:40 a

Re: Review Request 51222: RU, order Kafka right after Ranger and before HDFS in upgrade pack

2016-08-18 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51222/#review146142 --- Ship it! Ship It! - Jonathan Hurley On Aug. 18, 2016, 5:11 p

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
> On Сер. 18, 2016, 8:54 після полудня, Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py, > > line 71 > > > > > > Can we call kinit if Kerbe

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146134 --- Ship it! ambari-server/src/main/resources/common-services/ATLA

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-18 Thread Anita Jebaraj
> On Aug. 18, 2016, 9:11 p.m., Nate Cole wrote: > > What is the status of this review? If it has been pushed, please mark it > > as Submitted. The patch was reverted, I will close this review as discarded and open a new Review Request for the new patch. - Anita ---

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-18 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review146131 --- What is the status of this review? If it has been pushed, please

Re: Review Request 51222: RU, order Kafka right after Ranger and before HDFS in upgrade pack

2016-08-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51222/#review146132 --- ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/con

Review Request 51222: RU, order Kafka right after Ranger and before HDFS in upgrade pack

2016-08-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51222/ --- Review request for Ambari, Dmytro Grinenko, Jonathan Hurley, Nate Cole, and Naha

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146127 --- ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Dmytro Grinenko
> On Aug. 18, 2016, 8:22 p.m., Dmytro Grinenko wrote: > > Ship It! After fixing issue above. - Dmytro --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146124 -

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146124 --- Ship it! Ship It! - Dmytro Grinenko On Aug. 18, 2016, 7:42 p

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146123 --- ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146122 --- ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/ --- (Updated Сер. 18, 2016, 7:42 після полудня) Review request for Ambari, Alejandr

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
> On Сер. 18, 2016, 5:38 після полудня, Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py, > > line 71 > > > > > > Should not kinit call be c

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/ --- (Updated Сер. 18, 2016, 6:51 після полудня) Review request for Ambari, Alejandr

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/ --- (Updated Сер. 18, 2016, 6:47 після полудня) Review request for Ambari, Andrew O

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Sumit Mohanty
> On Aug. 18, 2016, 5:38 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py, > > line 71 > > > > > > Should not kinit call be conditiona

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
> On Сер. 18, 2016, 5:38 після полудня, Sumit Mohanty wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/metainfo.xml, > > line 34 > > > > > > I was expecting an kerberos.json change to ensure

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
> On Сер. 18, 2016, 5:38 після полудня, Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py, > > line 71 > > > > > > Should not kinit call be c

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
> On Сер. 18, 2016, 5:31 після полудня, Alejandro Fernandez wrote: > > ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py, > > line 94 > > > > > > If the unit test fails, we should remove it

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-18 Thread Anita Jebaraj
> On Aug. 18, 2016, 2:05 p.m., Robert Levas wrote: > > Ship It! > > Anita Jebaraj wrote: > Can you please help in pushing the code? > > Robert Levas wrote: > Committed to trunk > ``` > commit 712a12b6b3228d5bd532aa7a594544c8bd594d66 > Author: Anita Jebaraj > Date: Thu

Re: Review Request 50995: Collect JVM Heap, GC and thread pool metrics from Ambari Server and push to AMS

2016-08-18 Thread Li-Wei Tseng
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50995/ --- (Updated Aug. 18, 2016, 6:03 p.m.) Review request for Ambari, Aravindan Vijayan

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-18 Thread Robert Levas
> On Aug. 18, 2016, 10:05 a.m., Robert Levas wrote: > > Ship It! > > Anita Jebaraj wrote: > Can you please help in pushing the code? Committed to trunk ``` commit 712a12b6b3228d5bd532aa7a594544c8bd594d66 Author: Anita Jebaraj Date: Thu Aug 18 13:47:26 2016 -0400 ``` Committed to branch-

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146101 --- ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146099 --- ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regio

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146098 --- ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-18 Thread Anita Jebaraj
> On Aug. 18, 2016, 2:05 p.m., Robert Levas wrote: > > Ship It! Can you please help in pushing the code? - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51189/#review146078 --

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/ --- (Updated Сер. 18, 2016, 4:04 після полудня) Review request for Ambari, Andrew O

Review Request 51207: Wrong hostname in timeline.metrics.service.webapp.address breaks AMS HA

2016-08-18 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51207/ --- Review request for Ambari, Aravindan Vijayan and Sid Wagle. Bugs: AMBARI-18199

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/#review146086 --- Ship it! Ship It! - Dmytro Sen On Авг. 18, 2016, 3:35 п.п.,

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/ --- (Updated Сер. 18, 2016, 3:30 після полудня) Review request for Ambari, Dmitro L

Re: Review Request 51143: Alert on Atlas after adding it to a secure cluster as HBase table initialization fails

2016-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51143/ --- (Updated Сер. 18, 2016, 3:27 після полудня) Review request for Ambari, Dmitro L

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-18 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51189/#review146078 --- Ship it! Ship It! - Robert Levas On Aug. 18, 2016, 1:45 a.m.

Re: Review Request 51197: Zeppelin logsearch configuration typo

2016-08-18 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51197/#review146075 --- Ship it! Ship It! - Robert Nettleton On Aug. 18, 2016, 9:19

Re: Review Request 51203: Doc updates about ldap sync related properties

2016-08-18 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51203/#review146074 --- Ship it! Ship It! - Robert Nettleton On Aug. 18, 2016, 12:40

Review Request 51203: Doc updates about ldap sync related properties

2016-08-18 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51203/ --- Review request for Ambari, Jonathan Hurley, Laszlo Puskas, Sandor Magyari, and S

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-18 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51189/#review146073 --- Ship it! Ship It! - Di Li On Aug. 18, 2016, 5:45 a.m., Anita

Review Request 51201: Generate REST API docs with Swagger for Log Search

2016-08-18 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51201/ --- Review request for Ambari, Dharmesh Makwana, Miklos Gergely, Robert Nettleton, a

Re: Review Request 51199: Hsi-Hs2: Mondrian tests intermittently fail with "java.io.IOException: Connection reset by peer"

2016-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51199/#review146072 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 18, 2016, 9:2

Review Request 51199: Hsi-Hs2: Mondrian tests intermittently fail with "java.io.IOException: Connection reset by peer"

2016-08-18 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51199/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-18194 https:

Review Request 51198: Hsi-Hs2: Mondrian tests intermittently fail with "java.io.IOException: Connection reset by peer"

2016-08-18 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51198/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-18194 https:

Re: Review Request 51197: Zeppelin logsearch configuration typo

2016-08-18 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51197/#review146071 --- Ship it! Ship It! - Oliver Szabo On Aug. 18, 2016, 9:19 a.m.

Review Request 51197: Zeppelin logsearch configuration typo

2016-08-18 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51197/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. B