Re: Review Request 59877: Custom log4j.properties of Flume

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59877/#review178140 --- ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/

Re: Review Request 60078: AMBARI-21190 Populate urlauth filesystem schemes in Ranger hive plugin

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60078/#review178136 --- ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/config-u

Re: Review Request 60113: Exception needs to be handled properly for mail alert

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60113/#review178132 --- Ship it! Ship It! - Alejandro Fernandez On June 16, 2017, 2:

Re: Review Request 60050: AMBARI-21240. Some topology request data retained after host removed

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60050/#review178131 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60153/#review178130 --- Ship it! Ship It! - Alejandro Fernandez On June 16, 2017, 12

Re: Review Request 60098: Add Kerberos HTTP SPNEGO authentication support to Accumulo

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60098/#review178129 --- Ship it! Ship It! - Alejandro Fernandez On June 14, 2017, 9:

Re: Review Request 60152: Add "zeppelin.server.ssl.port" in Zeppelin's setting

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60152/#review178128 --- ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2

Re: Review Request 60151: Ability to auto-start component without server intervention

2017-06-16 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60151/#review178127 --- Ship it! Ship It! - Alejandro Fernandez On June 16, 2017, 8:

Re: Review Request 60164: Swagger resources generation for doc purpose should not be part of default maven profile

2017-06-16 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60164/#review178125 --- Received "+1" on apache jira: https://issues.apache.org/jira/brows

Review Request 60164: Swagger resources generation for doc purpose should not be part of default maven profile

2017-06-16 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60164/ --- Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Sumit Mohanty, a

Re: Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60153/#review178113 --- Ship it! Ship It! - Dmitro Lisnichenko On June 16, 2017, 3:4

Review Request 60156: AMBARI-21158. Eliminate Maven warnings

2017-06-16 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60156/ --- Review request for Ambari, Jonathan Hurley, Laszlo Puskas, Nate Cole, and Sebast

Re: Review Request 60050: AMBARI-21240. Some topology request data retained after host removed

2017-06-16 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60050/#review178110 --- Ship it! Ship It! - Sebastian Toader On June 16, 2017, 5:03

Re: Review Request 60113: Exception needs to be handled properly for mail alert

2017-06-16 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60113/#review178108 --- ambari-server/src/test/java/org/apache/ambari/server/state/servic

Re: Review Request 60050: AMBARI-21240. Some topology request data retained after host removed

2017-06-16 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60050/ --- (Updated June 16, 2017, 5:03 p.m.) Review request for Ambari, Sandor Magyari, S

Re: Review Request 60113: Exception needs to be handled properly for mail alert

2017-06-16 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60113/#review178107 --- Ship it! Ship It! - Aravindan Vijayan On June 16, 2017, 2:39

Re: Review Request 60113: Exception needs to be handled properly for mail alert

2017-06-16 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60113/ --- (Updated Июнь 16, 2017, 2:39 п.п.) Review request for Ambari, Aravindan Vijayan

Re: Review Request 60078: AMBARI-21190 Populate urlauth filesystem schemes in Ranger hive plugin

2017-06-16 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60078/#review178106 --- Ship it! Ship It! - Jonathan Hurley On June 14, 2017, 9:37 a

Re: Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60153/#review178105 --- Ship it! Ship It! - Robert Levas On June 16, 2017, 8:40 a.m.

Re: Review Request 60120: Remove metainfo latest url

2017-06-16 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60120/#review178104 --- Ship it! Ship It! - Jonathan Hurley On June 16, 2017, 9:07 a

Re: Review Request 60120: Remove metainfo latest url

2017-06-16 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60120/#review178103 --- Ship it! Ship It! - Nate Cole On June 16, 2017, 9:07 a.m., D

Re: Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Eugene Chekanskiy
> On June 16, 2017, 1:05 p.m., Dmitro Lisnichenko wrote: > > ambari-server/src/main/python/ambari_server/setupSecurity.py > > Line 276 (original) > > > > > > what magic number 4 means? no matter, it is removed anywa

Re: Review Request 60120: Remove metainfo latest url

2017-06-16 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60120/ --- (Updated June 16, 2017, 4:07 p.m.) Review request for Ambari, Jonathan Hurley a

Re: Review Request 60120: Remove metainfo latest url

2017-06-16 Thread Dmitro Lisnichenko
> On June 15, 2017, 6:11 p.m., Nate Cole wrote: > > There's more code than what you have changed: > > > > https://github.com/apache/ambari/blob/trunk/ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java#L144-L152 > > https://github.com/apache/ambari/blob/trunk/am

Re: Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60153/#review178100 --- ambari-server/src/main/python/ambari_server/setupSecurity.py Line

Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60153/ --- Review request for Ambari, Dmitro Lisnichenko, Robert Levas, and Vitalyi Brodets

Re: Review Request 60130: parquet-logging.properties is missing in hive conf when HDP is upgraded

2017-06-16 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60130/#review178096 --- Ship it! Ship It! - Robert Levas On June 15, 2017, 4:08 p.m.

Re: Review Request 60098: Add Kerberos HTTP SPNEGO authentication support to Accumulo

2017-06-16 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60098/#review178095 --- Ship it! Ship It! - Dmytro Sen On Июнь 14, 2017, 9:24 п.п.,

Re: Review Request 60114: Remove YARN client mode option from WFM

2017-06-16 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60114/#review178094 --- contrib/views/wfmanager/src/main/resources/ui/app/components/flow

Re: Review Request 60152: Add "zeppelin.server.ssl.port" in Zeppelin's setting

2017-06-16 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60152/#review178093 --- Ship it! Ship It! - venkat sairam On June 16, 2017, 9:28 a.m

Review Request 60152: Add "zeppelin.server.ssl.port" in Zeppelin's setting

2017-06-16 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60152/ --- Review request for Ambari, Attila Doroszlai, Alejandro Fernandez, DIPAYAN BHOWMI

Review Request 60151: Ability to auto-start component without server intervention

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

Re: Review Request 60131: update yarn-env.xml in HDP2.6 for YARN sinks' HTTP SPNEGO Authentication

2017-06-16 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60131/#review178085 --- Ship it! Ship It! - Dmytro Sen On Июнь 15, 2017, 7:38 п.п.,