Re: Review Request 50822: Enforce granular role-based access control for log search functions

2016-08-05 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50822/#review144897 --- Ship it! Ship It! - Oliver Szabo On Aug. 4, 2016, 11:04 p.m.

Review Request 50840: Ldap sync filter logged out after the ldapsearch query happens

2016-08-05 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50840/ --- Review request for Ambari, Laszlo Puskas, Robert Levas, and Sebastian Toader. B

Re: Review Request 50840: Ldap sync filter logged out after the ldapsearch query happens

2016-08-05 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50840/#review144900 --- Ship it! Ship It! - Laszlo Puskas On Aug. 5, 2016, 9:26 a.m.

Re: Review Request 50840: Ldap sync filter logged out after the ldapsearch query happens

2016-08-05 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50840/#review144901 --- Ship it! Ship It! - Sebastian Toader On Aug. 5, 2016, 11:26

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

2016-08-05 Thread Robert Levas
> On Aug. 3, 2016, 5:17 p.m., Robert Levas wrote: > > Ship It! Committed to trunk ``` commit fb2c12964ad26d6460b0d73a00a9750e0bf0442d Author: Anita Jebaraj Date: Fri Aug 5 06:20:49 2016 -0400 ``` Committed to branch-2.4 ``` commit 79a8296a5883fb04a91f3ef8472d193deacd987f Author: Anita Jebara

Review Request 50847: Not able to proceed with RU downgrade due to spark clients failing to install

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

Re: Review Request 50847: Not able to proceed with RU downgrade due to spark clients failing to install

2016-08-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50847/#review144915 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 5, 2016, 11:2

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-05 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50753/#review144920 --- Ship it! This is an AmbariEvent which has it's own EventBus; ho

Re: Review Request 50297: AMBARI-17788: Refactor spooler code in OutputHDFSFile to be reusable for OutputS3File

2016-08-05 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50297/#review144922 --- committed to trunk: c63b34cc2d02d58e4b76ac29c50e812a10eb3e6c now y

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-05 Thread Laszlo Puskas
> On Aug. 5, 2016, 12:36 p.m., Jonathan Hurley wrote: > > This is an AmbariEvent which has it's own EventBus; however the bus is an > > asynchronous, single-threaded bus. This means that it's possible for a > > heartbeat to be received in between when this event is broadcast and when > > it's

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-05 Thread Laszlo Puskas
> On Aug. 5, 2016, 12:36 p.m., Jonathan Hurley wrote: > > This is an AmbariEvent which has it's own EventBus; however the bus is an > > asynchronous, single-threaded bus. This means that it's possible for a > > heartbeat to be received in between when this event is broadcast and when > > it's

Review Request 50849: HBase start failing with API

2016-08-05 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50849/ --- Review request for Ambari and Andrew Onischuk. Bugs: AMBARI-18038 https://i

Re: Review Request 50849: HBase start failing with API

2016-08-05 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50849/#review144925 --- Ship it! Ship It! - Andrew Onischuk On Aug. 5, 2016, 1:03 p.

Re: Review Request 50822: Enforce granular role-based access control for log search functions

2016-08-05 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50822/#review144926 --- Ship it! Ship It! - Robert Nettleton On Aug. 4, 2016, 11:04

Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav Kulshreshth

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/#review144929 --- Ship it! Ship It! - Rohit Choudhary On Aug. 5, 2016, 1:26 p.

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/#review144931 --- Ship it! Ship It! - Pallav Kulshreshtha On Aug. 5, 2016, 1:2

Re: Review Request 50753: Cleared cached resources from ambari-server on host removal

2016-08-05 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50753/ --- (Updated Aug. 5, 2016, 2:39 p.m.) Review request for Ambari, Jonathan Hurley, R

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/#review144928 --- Can you add some unit tests for DateJsonDeserializer? ambari-ser

Re: Review Request 49935: AMBARI-17633 yarn.nodemanager.remote-app-log-dir should be added stickybit.

2016-08-05 Thread Jayush Luniya
> On Aug. 5, 2016, 6:02 a.m., Masahiro Tanaka wrote: > > Thank you for reviewing! Could anyone commit this? Done. Please close review request. - Jayush --- This is an automatically generated e-mail. To reply, visit: https://reviews.apac

Review Request 50860: ambari-server upgrade should report all the config properties it added/modified

2016-08-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50860/ --- Review request for Ambari, Andrew Onischuk, Dmytro Sen, Sumit Mohanty, and Sid W

Re: Review Request 50860: ambari-server upgrade should report all the config properties it added/modified

2016-08-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50860/#review144959 --- For Unit tests i will create separate jira. Because ConfigurationT

Re: Review Request 50860: ambari-server upgrade should report all the config properties it added/modified

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

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/ --- (Updated Aug. 5, 2016, 6:19 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Gaurav Nagar
> On Aug. 5, 2016, 2:58 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/DateJsonDeserializer.java, > > line 42 > > > > > > From http://docs.oracle.com/javase/7/docs/api/java/tex

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/#review144973 --- Fix it, then Ship it! Thanks for making the changes and ading t

Review Request 50864: Zeppelin to participate in RU/EU orchestration and advertise a version

2016-08-05 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50864/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Rohit Choudhar

Re: Review Request 50864: Zeppelin to participate in RU/EU orchestration and advertise a version

2016-08-05 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50864/#review144978 --- Ship it! Ship It! - Rohit Choudhary On Aug. 5, 2016, 8:28 p.

Review Request 50865: Starting a Component After Pausing An Upgrade Can Take 9 Minutes

2016-08-05 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50865/ --- Review request for Ambari, Alejandro Fernandez, Robert Levas, and Robert Nettlet

Re: Review Request 50865: Starting a Component After Pausing An Upgrade Can Take 9 Minutes

2016-08-05 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50865/#review144980 --- Ship it! Ship It! - Robert Nettleton On Aug. 5, 2016, 8:37 p

Re: Review Request 48766: MySQL service status needs to be more robust

2016-08-05 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48766/ --- (Updated Aug. 5, 2016, 9:49 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 48766: MySQL service status needs to be more robust

2016-08-05 Thread Juanjo Marron
> On Aug. 3, 2016, 12:33 a.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/mysql_service.py, > > line 36 > > > > > > What happens if pid_file is Non

Re: Review Request 50865: Starting a Component After Pausing An Upgrade Can Take 9 Minutes

2016-08-05 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50865/#review144995 --- Ship it! Ship It! - Robert Levas On Aug. 5, 2016, 4:37 p.m.,

Re: Review Request 50864: Zeppelin to participate in RU/EU orchestration and advertise a version

2016-08-05 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50864/#review145000 --- ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/upgrade-

Review Request 50870: Introduce validation of hostgroup mapping for active/standby namenode

2016-08-05 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50870/ --- Review request for Ambari, Balázs Bence Sári and Robert Nettleton. Bugs: AMBARI

Re: Review Request 49935: AMBARI-17633 yarn.nodemanager.remote-app-log-dir should be added stickybit.

2016-08-05 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49935/ --- (Updated Aug. 6, 2016, 10:17 a.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 49935: AMBARI-17633 yarn.nodemanager.remote-app-log-dir should be added stickybit.

2016-08-05 Thread Masahiro Tanaka
> On Aug. 5, 2016, 3:02 p.m., Masahiro Tanaka wrote: > > Thank you for reviewing! Could anyone commit this? > > Jayush Luniya wrote: > Done. Please close review request. Thnak you! - Masahiro --- This is an automatically generated

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/ --- (Updated Aug. 6, 2016, 6:35 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,