Re: Review Request 61169: Remove environment variables from zeppelin-env.sh

2017-07-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61169/#review181547 --- Ship it! Ship It! - Gaurav Nagar On July 27, 2017, 5:09

Re: Review Request 61176: Apply zeppelin.server.kerberos.principal related changes to 0.6.0.3.0

2017-07-27 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61176/#review181546 --- Ship it! Ship It! - venkat sairam On July 27, 2017, 9:09

Re: Review Request 61162: Ooozie quicklink configuration doesn't work with https

2017-07-27 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61162/#review181540 --- Ship it! Ship It! - Miklos Gergely On July 26, 2017, 9:22

Re: Review Request 61169: Remove environment variables from zeppelin-env.sh

2017-07-27 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61169/#review181545 --- Ship it! Ship It! - venkat sairam On July 27, 2017, 5:09

Re: Review Request 61162: Ooozie quicklink configuration doesn't work with https

2017-07-27 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61162/#review181541 --- Ship it! Ship It! - Sebastian Toader On July 26, 2017,

Re: Review Request 61163: testBadCredential UT fails

2017-07-27 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61163/#review181542 --- Ship it! Ship It! - Sebastian Toader On July 26, 2017,

Re: Review Request 61107: Migrate custom extension support

2017-07-27 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/#review181544 --- Ship it! Ship It! - Andrew Onischuk On July 26, 2017, 9:56

Review Request 61176: Apply zeppelin.server.kerberos.principal related changes to 0.6.0.3.0

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

Re: Review Request 61140: Ambari REST API to support setting rack info during add host

2017-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61140/#review181561 --- Fix it, then Ship it! Ship It!

Re: Review Request 61146: Replace Hard Coded stack-select Structures

2017-07-27 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61146/#review181559 --- Ship it! Ship It! - Dmytro Grinenko On July 26, 2017, 7:04

Re: Review Request 61162: Ooozie quicklink configuration doesn't work with https

2017-07-27 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61162/#review181549 --- Ship it! Ship It! - Andrii Tkach On July 26, 2017, 9:22

Review Request 61177: Don't set restart indicators for NN/RM during DN/NM deletion

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

Re: Review Request 61140: Ambari REST API to support setting rack info during add host

2017-07-27 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61140/#review181553 --- Ship it! Ship It! - Attila Doroszlai On July 26, 2017, 5:42

Re: Review Request 61146: Replace Hard Coded stack-select Structures

2017-07-27 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61146/#review181558 --- Ship it! Ship It! - Dmitro Lisnichenko On July 26, 2017,

Re: Review Request 61140: Ambari REST API to support setting rack info during add host

2017-07-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61140/#review181555 --- Ship it! Ship It! - Laszlo Puskas On July 26, 2017, 3:42

Re: Review Request 61140: Ambari REST API to support setting rack info during add host

2017-07-27 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61140/#review181550 --- Ship it! Ship It! - Attila Magyar On July 26, 2017, 3:42

Re: Review Request 61177: Don't set restart indicators for NN/RM during DN/NM deletion

2017-07-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61177/ --- (Updated Июль 27, 2017, 1:26 п.п.) Review request for Ambari, Sumit Mohanty,

Re: Review Request 61140: Ambari REST API to support setting rack info during add host

2017-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61140/#review181564 --- Ship it! Ship It! - Sumit Mohanty On July 26, 2017, 3:42

Re: Review Request 61140: Ambari REST API to support setting rack info during add host

2017-07-27 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61140/ --- (Updated July 27, 2017, 5:25 p.m.) Review request for Ambari, Attila

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61178/#review181566 --- Hello Attila, The issue with the alert (2nd part of the issue),

Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61178/ --- Review request for Ambari, Alejandro Fernandez, Eric Yang, Jonathan Hurley,

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Attila Doroszlai
> On July 27, 2017, 4:38 p.m., Di Li wrote: > > Hello Attila, > > > > The issue with the alert (2nd part of the issue), was BI stack has its own > > sparks thrift server alert that check hardcoded port 4038. HDP stack uses a > > different alert for the same purpose. > > > > This manifests

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Attila Doroszlai
> On July 27, 2017, 4:38 p.m., Di Li wrote: > > Hello Attila, > > > > The issue with the alert (2nd part of the issue), was BI stack has its own > > sparks thrift server alert that check hardcoded port 4038. HDP stack uses a > > different alert for the same purpose. > > > > This manifests

Re: Review Request 61140: Ambari REST API to support setting rack info during add host

2017-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61140/#review181573 --- Ship it! Ship It! - Robert Levas On July 27, 2017, 11:25

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Di Li
> On July 27, 2017, 2:38 p.m., Di Li wrote: > > Hello Attila, > > > > The issue with the alert (2nd part of the issue), was BI stack has its own > > sparks thrift server alert that check hardcoded port 4038. HDP stack uses a > > different alert for the same purpose. > > > > This manifests

Re: Review Request 61177: Don't set restart indicators for NN/RM during DN/NM deletion

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61177/#review181576 --- Ship it! Ship It! - Sid Wagle On July 27, 2017, 1:26 p.m.,

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61178/#review181565 ---

Re: Review Request 60943: Log Search Solr output properties should be provided by the Config API

2017-07-27 Thread Miklos Gergely
> On July 26, 2017, 6:43 p.m., Robert Nettleton wrote: > > ambari-logsearch/ambari-logsearch-config-api/src/main/java/org/apache/ambari/logsearch/config/api/LogSearchConfig.java > > Lines 67 (patched) > > > > > >

Re: Review Request 61162: Ooozie quicklink configuration doesn't work with https

2017-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61162/#review181572 --- Ship it! Ship It! - Robert Levas On July 26, 2017, 5:22

Re: Review Request 61146: Replace Hard Coded stack-select Structures

2017-07-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61146/#review181578 --- Ship it! Ship It! - Nate Cole On July 26, 2017, 3:04 p.m.,

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Attila Doroszlai
> On July 27, 2017, 4:35 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/config-upgrade.xml > > Lines 268 (patched) > > > > > > Can it be deleted unconditionally or

Re: Review Request 61107: Migrate custom extension support

2017-07-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/ --- (Updated Липень 27, 2017, 4:09 після полудня) Review request for Ambari,

Re: Review Request 61186: Service repos are not updated with "latest" url in repoinfo.xml

2017-07-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61186/#review181580 --- Ship it! Ship It! - Vitalyi Brodetskyi On Липень 27, 2017,

Re: Review Request 61132: Agent hangs when server is restarted during connection stage

2017-07-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61132/#review181595 --- Ship it! Ship It! - Alejandro Fernandez On July 26, 2017,

Re: Review Request 61186: Service repos are not updated with "latest" url in repoinfo.xml

2017-07-27 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61186/#review181581 --- Ship it! Ship It! - Dmytro Grinenko On July 27, 2017, 5:05

Re: Review Request 61186: Service repos are not updated with "latest" url in repoinfo.xml

2017-07-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61186/#review181583 --- Ship it! Ship It! - Dmytro Sen On Июль 27, 2017, 5:05 п.п.,

Review Request 61186: Service repos are not updated with "latest" url in repoinfo.xml

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

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61168/#review181597 --- Fix it, then Ship it!

Re: Review Request 61107: Migrate custom extension support

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/#review181629 --- Ship it! Ship It! - Sid Wagle On July 28, 2017, 12:08 a.m.,

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Robert Levas
> On July 27, 2017, 3:55 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FixAuthToLocalMappingAction.java > > Lines 36-37 (patched) > > > > > > There may be

Re: Review Request 61083: AMBARI-21563 : Copy Hadoop core-site.xml to Atlas conf when hadoop is configured for LDAP/AD.

2017-07-27 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61083/ --- (Updated July 27, 2017, 6:49 p.m.) Review request for Ambari, Alejandro

Re: Review Request 61083: AMBARI-21563 : Copy Hadoop core-site.xml to Atlas conf when hadoop is configured for LDAP/AD.

2017-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61083/#review181603 --- Ship it! Ship It! - Sumit Mohanty On July 27, 2017, 6:49

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61168/#review181606 ---

Re: Review Request 61083: AMBARI-21563 : Copy Hadoop core-site.xml to Atlas conf when hadoop is configured for LDAP/AD.

2017-07-27 Thread Vishal Suvagia via Review Board
> On July 25, 2017, 8:29 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py > > Lines 210 (patched) > > > > > > Multiple copies of core-site

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61168/ --- (Updated July 27, 2017, 8:22 p.m.) Review request for Ambari, Attila

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61168/ --- (Updated July 27, 2017, 8:23 p.m.) Review request for Ambari, Attila

Re: Review Request 61083: AMBARI-21563 : Copy Hadoop core-site.xml to Atlas conf when hadoop is configured for LDAP/AD.

2017-07-27 Thread Vishal Suvagia via Review Board
> On July 24, 2017, 4:22 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py > > Lines 210 (patched) > > > > > > Why do we need to delete? > >

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Attila Doroszlai
> On July 27, 2017, 9:43 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/config-upgrade.xml > > Lines 268 (patched) > > > > > > Is this deleting the config

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

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

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Sid Wagle
> On July 27, 2017, 7:55 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FixAuthToLocalMappingAction.java > > Lines 36-37 (patched) > > > > > > There may be

Re: Review Request 61203: AMBARI-21593 : AMS stopped after RU [AMS distributed mode with 2 collectors]

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61203/#review181647 ---

Re: Review Request 61203: AMBARI-21593 : AMS stopped after RU [AMS distributed mode with 2 collectors]

2017-07-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61203/#review181646 ---

Review Request 61203: AMBARI-21593 : AMS stopped after RU [AMS distributed mode with 2 collectors]

2017-07-27 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61203/ --- Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. Bugs:

Re: Review Request 61107: Migrate custom extension support

2017-07-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/ --- (Updated Липень 28, 2017, північ) Review request for Ambari, Andrew Onischuk,

Re: Review Request 61107: Migrate custom extension support

2017-07-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/ --- (Updated Липень 28, 2017, 12:08 до полудня) Review request for Ambari, Andrew

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61168/ --- (Updated July 27, 2017, 9:36 p.m.) Review request for Ambari, Attila

Re: Review Request 61107: Migrate custom extension support

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

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Alejandro Fernandez
> On July 27, 2017, 7:43 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/config-upgrade.xml > > Lines 268 (patched) > > > > > > Is this deleting the config

Review Request 61197: Log Feeder properties should be handled by one class

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

Review Request 61201: AMBARI-21594. MultiEverything : Add Servicegroup as a subresource of Cluster.

2017-07-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61201/ --- Review request for Ambari, Jayush Luniya and Madhuvanthi Radhakrishnan. Bugs:

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61168/ --- (Updated July 28, 2017, 4:14 a.m.) Review request for Ambari, Attila

Re: Review Request 61168: Hive-Service check failing in post EU validation (BI-HDP)

2017-07-27 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61168/ --- (Updated July 28, 2017, 4:16 a.m.) Review request for Ambari, Attila