Re: Review Request 56022: Content of yarn-env.sh on host is not same as in the downloaded config file from Ambari UI

2017-01-27 Thread Sebastian Toader
> On Jan. 27, 2017, 7:31 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration/yarn-env.xml, > > line 157 > > > > > > Do existing clusters with HDP 2.3 or

Re: Review Request 56022: Content of yarn-env.sh on host is not same as in the downloaded config file from Ambari UI

2017-01-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56022/#review163309 ---

Re: Review Request 56022: Content of yarn-env.sh on host is not same as in the downloaded config file from Ambari UI

2017-01-27 Thread Sebastian Toader
> On Jan. 27, 2017, 8 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration/yarn-env.xml, > > line 157 > > > > > > In that case, this should belong in

Re: Review Request 56022: Content of yarn-env.sh on host is not same as in the downloaded config file from Ambari UI

2017-01-27 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56022/ --- (Updated Jan. 27, 2017, 7:11 p.m.) Review request for Ambari, Laszlo Puskas,

Review Request 56033: Remove white border from the content area in popup wizards

2017-01-27 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56033/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Vivek Ratnavel

Re: Review Request 56015: Ambari upgrade to Ambari 2.5 causes start DB check failure on Zeppelin configs

2017-01-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56015/#review163305 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 27, 2017,

Re: Review Request 56015: Ambari upgrade to Ambari 2.5 causes start DB check failure on Zeppelin configs

2017-01-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56015/#review163306 ---

Review Request 56031: Hive2 View - Switching tabs after executing query loses state

2017-01-27 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56031/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 56029: AMBARI-19751: Credential Store params should be in the command json for RESTART

2017-01-27 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56029/#review163307 --- Ship it! Ship It! - Miklos Gergely On Jan. 27, 2017, 6:12

Review Request 56029: AMBARI-19751: Credential Store params should be in the command json for RESTART

2017-01-27 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56029/ --- Review request for Ambari, Miklos Gergely and Sumit Mohanty. Bugs:

Review Request 56022: Content of yarn-env.sh on host is not same as in the downloaded config file from Ambari UI

2017-01-27 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56022/ --- Review request for Ambari, Laszlo Puskas, Robert Levas, and Sebastian Toader.

Re: Review Request 56022: Content of yarn-env.sh on host is not same as in the downloaded config file from Ambari UI

2017-01-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56022/#review163303 ---

Review Request 56030: Hive View 2.0 execute button needs some feedback

2017-01-27 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56030/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 56033: Remove white border from the content area in popup wizards

2017-01-27 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56033/#review163321 --- Ship it! Ship It! - Vivek Ratnavel Subramanian On Jan. 27,

Re: Review Request 56033: Remove white border from the content area in popup wizards

2017-01-27 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56033/#review163323 --- Does this cover modal popups for all wizards ? Has this patch

Re: Review Request 56033: Remove white border from the content area in popup wizards

2017-01-27 Thread Xi Wang
> On Jan. 27, 2017, 12:11 p.m., Jaimin Jetly wrote: > > Does this cover modal popups for all wizards ? > > Has this patch been tested with wizards ? Yes, covered all wizards in the popup. Tested and you can see scrennshots in the JIRA. - Xi

Review Request 56050: Kerberizing PERF cluster fails since cannot find principal name config in dummy.py

2017-01-27 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56050/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit

Review Request 56039: Add common log rotation settings to hive log4j2

2017-01-27 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56039/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit

Re: Review Request 56039: Add common log rotation settings to hive log4j2

2017-01-27 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56039/ --- (Updated Jan. 27, 2017, 9:51 p.m.) Review request for Ambari, Alejandro

Re: Review Request 56039: Add common log rotation settings to hive log4j2

2017-01-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56039/#review163353 --- Ship it! Ship It! - Sumit Mohanty On Jan. 27, 2017, 9:51

Re: Review Request 56033: Remove white border from the content area in popup wizards

2017-01-27 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56033/#review163360 --- Ship it! Ship It! - Jaimin Jetly On Jan. 27, 2017, 7:25

Re: Review Request 56030: Hive View 2.0 execute button needs some feedback

2017-01-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56030/#review163386 --- Ship it! Ship It! - Gaurav Nagar On Jan. 27, 2017, 6:20

Re: Review Request 56031: Hive2 View - Switching tabs after executing query loses state

2017-01-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56031/#review163385 --- Ship it! Ship It! - Gaurav Nagar On Jan. 27, 2017, 6:42

Re: Review Request 56021: AMBARI-19743. Agent registration fails with non-root agent user

2017-01-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56021/#review163382 --- Ship it! Ship It! - Sumit Mohanty On Jan. 27, 2017, 2:38

Re: Review Request 56050: Kerberizing PERF cluster fails since cannot find principal name config in dummy.py

2017-01-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56050/#review163383 --- Ship it! Ship It! - Sid Wagle On Jan. 28, 2017, 2:12 a.m.,

Re: Review Request 55986: Extend security.json of Infra Solr as a configuration property in Ambari

2017-01-27 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55986/#review163240 --- Ship it! Ship It! - Miklos Gergely On Jan. 26, 2017, 5:48

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Attila Doroszlai
> On Jan. 26, 2017, 9:17 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/resources/scripts/post-user-creation-hook.sh, line 34 > > > > > > I think using `$3` would be better, unless this one will remain the

Review Request 56010: AMBARI-19709:XML generated for bundle's is not as per oozie xsd

2017-01-27 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56010/ --- Review request for Ambari, Gaurav Nagar, Nitiraj Rathore, and Pallav

Review Request 56011: AMBARI-19739 Stack changes to allow : Hive Interactive plugin to communicate with external Ranger

2017-01-27 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56011/ --- Review request for Ambari, Gautam Borad, Sumit Mohanty, and Velmurugan

Re: Review Request 56011: AMBARI-19739 Stack changes to allow : Hive Interactive plugin to communicate with external Ranger

2017-01-27 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56011/#review163243 --- Ship it! Ship It! - Gautam Borad On Jan. 27, 2017, 10:01

Review Request 56013: Ambari Server Unit Test failure on branch-2.5/trunk for testUpdateConfigForceSecurityEnabled

2017-01-27 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56013/ --- Review request for Ambari, Dmitro Lisnichenko and Robert Levas. Bugs:

Re: Review Request 56013: Ambari Server Unit Test failure on branch-2.5/trunk for testUpdateConfigForceSecurityEnabled

2017-01-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56013/#review163246 --- Ship it! Ship It! - Robert Levas On Jan. 27, 2017, 6:25

Re: Review Request 55985: Atlas deployment via Ambari should configure Zookeeper ACLs and Auth scheme

2017-01-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55985/#review163247 --- Ship it! Ship It! - Robert Levas On Jan. 26, 2017, 11:38

Re: Review Request 56013: Ambari Server Unit Test failure on branch-2.5/trunk for testUpdateConfigForceSecurityEnabled

2017-01-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56013/#review163248 --- Ship it! Ship It! - Dmitro Lisnichenko On Jan. 27, 2017,

Review Request 56014: User preference like opened tabs/work in progress should not be available to a different user

2017-01-27 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56014/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Pallav Kulshreshtha, and

Re: Review Request 56015: Ambari upgrade to Ambari 2.5 causes start DB check failure on Zeppelin configs

2017-01-27 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56015/#review163254 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 27, 2017,

Re: Review Request 56015: Ambari upgrade to Ambari 2.5 causes start DB check failure on Zeppelin configs

2017-01-27 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56015/#review163255 --- Ship it! Ship It! - Rohit Choudhary On Jan. 27, 2017, 12:43

Re: Review Request 56015: Ambari upgrade to Ambari 2.5 causes start DB check failure on Zeppelin configs

2017-01-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56015/#review163257 --- Ship it! Ship It! - Robert Levas On Jan. 27, 2017, 7:43

Re: Review Request 56014: User preference like opened tabs/work in progress should not be available to a different user

2017-01-27 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56014/ --- (Updated Jan. 27, 2017, 1:17 p.m.) Review request for Ambari, belliraj hb,

Re: Review Request 56014: User preference like opened tabs/work in progress should not be available to a different user

2017-01-27 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56014/#review163262 --- Ship it! Ship It! - Pallav Kulshreshtha On Jan. 27, 2017,

Re: Review Request 56014: User preference like opened tabs/work in progress should not be available to a different user

2017-01-27 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56014/#review163263 --- Ship it! Ship It! - Renjith Kamath On Jan. 27, 2017, 1:17

Re: Review Request 56014: User preference like opened tabs/work in progress should not be available to a different user

2017-01-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56014/#review163264 --- Ship it! Ship It! - Gaurav Nagar On Jan. 27, 2017, 1:17

Re: Review Request 55986: Extend security.json of Infra Solr as a configuration property in Ambari

2017-01-27 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55986/#review163266 --- Ship it! Ship It! - Robert Nettleton On Jan. 26, 2017, 5:48

Review Request 56020: Ambari HDFS Metric alerts turns to UNKNOWN status with error "argument of type 'NoneType' is not iterable"

2017-01-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56020/ --- Review request for Ambari, Aravindan Vijayan, Sid Wagle, and Vitalyi Brodetskyi.

Review Request 56021: AMBARI-19743. Agent registration fails with non-root agent user

2017-01-27 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56021/ --- Review request for Ambari, Sebastian Toader, Sid Wagle, and Vitalyi Brodetskyi.

Re: Review Request 56021: AMBARI-19743. Agent registration fails with non-root agent user

2017-01-27 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56021/#review163267 --- Ship it! Ship It! - Sebastian Toader On Jan. 27, 2017, 3:38

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 2:59 p.m.) Review request for Ambari, Attila

Re: Review Request 55505: AMBARI-19524 : Use stack-level Atlas conf directory instead of hard-coded /etc/atlas/conf path

2017-01-27 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55505/ --- (Updated Jan. 27, 2017, 3:05 p.m.) Review request for Ambari, Alejandro

Re: Review Request 55505: AMBARI-19524 : Use stack-level Atlas conf directory instead of hard-coded /etc/atlas/conf path

2017-01-27 Thread Vishal Suvagia
> On Jan. 25, 2017, 10:07 p.m., Alejandro Fernandez wrote: > > ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py, > > line 35 > > > > > > You can retrieve the stack root from the cluster-env

Review Request 56023: Upgrade default JDK installed by Ambari to be >8u100

2017-01-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56023/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Dmytro Sen.

Re: Review Request 56023: Upgrade default JDK installed by Ambari to be >8u100

2017-01-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56023/#review163270 --- Ship it! Ship It! - Dmytro Sen On Янв. 27, 2017, 3:14 п.п.,

Re: Review Request 56023: Upgrade default JDK installed by Ambari to be >8u100

2017-01-27 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56023/#review163271 --- Ship it! Ship It! - Dmytro Grinenko On Jan. 27, 2017, 3:14

Re: Review Request 56020: Ambari HDFS Metric alerts turns to UNKNOWN status with error "argument of type 'NoneType' is not iterable"

2017-01-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56020/#review163272 --- Ship it! Ship It! - Vitalyi Brodetskyi On Січ. 27, 2017,

Re: Review Request 56020: Ambari HDFS Metric alerts turns to UNKNOWN status with error "argument of type 'NoneType' is not iterable"

2017-01-27 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56020/#review163273 ---

Re: Review Request 55981: Log Search portal not working if only solr needs SSL connection

2017-01-27 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55981/ --- (Updated Jan. 27, 2017, 3:24 p.m.) Review request for Ambari, Oliver Szabo and

Re: Review Request 55986: Extend security.json of Infra Solr as a configuration property in Ambari

2017-01-27 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55986/ --- (Updated Jan. 27, 2017, 3:32 p.m.) Review request for Ambari, Miklos Gergely

Re: Review Request 56023: Upgrade default JDK installed by Ambari to be >8u100

2017-01-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56023/#review163275 --- Ship it! Ship It! - Dmitro Lisnichenko On Jan. 27, 2017,

Re: Review Request 55981: Log Search portal not working if only solr needs SSL connection

2017-01-27 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55981/#review163276 --- Ship it! Ship It! - Robert Nettleton On Jan. 27, 2017, 3:24

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 4:02 p.m.) Review request for Ambari, Attila

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/#review163280 --- Fix it, then Ship it!

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 4:12 p.m.) Review request for Ambari, Attila

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 4:12 p.m.) Review request for Ambari, Attila

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/#review163282 --- Ship it! Ship It! - Sebastian Toader On Jan. 27, 2017, 5:12

Re: Review Request 56020: Ambari HDFS Metric alerts turns to UNKNOWN status with error "argument of type 'NoneType' is not iterable"

2017-01-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56020/#review163285 --- Ship it! Ship It! - Sid Wagle On Jan. 27, 2017, 2:22 p.m.,

Re: Review Request 56020: Ambari HDFS Metric alerts turns to UNKNOWN status with error "argument of type 'NoneType' is not iterable"

2017-01-27 Thread Sid Wagle
> On Jan. 27, 2017, 3:22 p.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/ambari_commons/ambari_metrics_helper.py, line > > 49 > > > > > > load_properties_from_file doesn't provide possibility to check

Re: Review Request 55935: When hiveserver2 LDAP authentication is enabled hive.server2.authentication.ldap.url property gets exported with hardcoded hostname

2017-01-27 Thread Robert Nettleton
> On Jan. 26, 2017, 4:25 p.m., Robert Nettleton wrote: > > Ship It! > > Amruta Borkar wrote: > Hello Robert, > Could you help me push this to trunk? > Thank you. > > Robert Nettleton wrote: > Hi Amruta, > > Sure, I'll try to push this this morning. > >