Re: Review Request 64614: AMBARI-22164. Fix wrong assignment due to leftover parameter name

2017-12-14 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64614/#review193844 --- Ship it! Ship It! - Madhuvanthi Radhakrishnan On Dec. 14,

Re: Review Request 64606: Fix unit tests in feature branch to make them workable

2017-12-14 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64606/#review193800 --- Ship it! Ship It! - Attila Doroszlai On Dec. 14, 2017, 2:46

Review Request 64607: Ambari Infra Manager: solr data exporting jobs and integration test environment

2017-12-14 Thread Krisztian Kasa
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64607/ --- Review request for Ambari, Miklos Gergely, Oliver Szabo, and Sid Wagle. Bugs:

Re: Review Request 64607: Ambari Infra Manager: solr data exporting jobs and integration test environment

2017-12-14 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64607/#review193796 --- ambari-infra/ambari-infra-manager-it/pom.xml Lines 111 (patched)

Review Request 64606: Fix unit tests in feature branch to make them workable

2017-12-14 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64606/ --- Review request for Ambari, Attila Doroszlai and Dmytro Sen. Bugs: AMBARI-22614

Review Request 64611: AMBARI-22196. yum repo file contains parsing errors due to initial whitespace

2017-12-14 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64611/ --- Review request for Ambari, Balázs Bence Sári, Jayush Luniya, and Swapan

Review Request 64613: AMBARI-22249. Fix AmbariContext.serviceGroupDependencyResourceProvider initialization

2017-12-14 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64613/ --- Review request for Ambari, Balázs Bence Sári, Jayush Luniya, Swapan Shridhar,

Review Request 64614: AMBARI-22164. Fix wrong assignment due to leftover parameter name

2017-12-14 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64614/ --- Review request for Ambari, Balázs Bence Sári, Jayush Luniya, Madhuvanthi

Re: Review Request 64542: Kerberos support for OneFS

2017-12-14 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64542/#review193804 --- Ship it! Ship It! - Robert Levas On Dec. 14, 2017, 6:26

Re: Review Request 64594: AMBARI-22649. Library for querying clusterSettings and stackSettings for its contents in command*.json

2017-12-14 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64594/#review193820 ---

Re: Review Request 64594: AMBARI-22649. Library for querying clusterSettings and stackSettings for its contents in command*.json

2017-12-14 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64594/#review193825 --- Fix it, then Ship it!

Re: Review Request 64594: AMBARI-22649. Library for querying clusterSettings and stackSettings for its contents in command*.json

2017-12-14 Thread Swapan Shridhar
> On Dec. 14, 2017, 7:01 p.m., Madhuvanthi Radhakrishnan wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/cluster_settings.py > > Lines 45 (patched) > > > > > > Hi, > > Eventually

Re: Review Request 64594: AMBARI-22649. Library for querying clusterSettings and stackSettings for its contents in command*.json

2017-12-14 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64594/ --- (Updated Dec. 14, 2017, 9:02 p.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 64594: AMBARI-22649. Library for querying clusterSettings and stackSettings for its contents in command*.json

2017-12-14 Thread Swapan Shridhar
> On Dec. 14, 2017, 5:37 p.m., Jayush Luniya wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_settings.py > > Lines 24 (patched) > > > > > > STACK_NAME_SETTING_KEY or

Re: Review Request 64637: Livy/Livy2 Unable To Start Due to Address Already In Use

2017-12-14 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64637/#review193874 ---

Review Request 64637: Livy/Livy2 Unable To Start Due to Address Already In Use

2017-12-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64637/ --- Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole.

Re: Review Request 64637: Livy/Livy2 Unable To Start Due to Address Already In Use

2017-12-14 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64637/#review193872 --- Ship it! Ship It! - Jayush Luniya On Dec. 15, 2017, 3:03

Re: Review Request 64637: Livy/Livy2 Unable To Start Due to Address Already In Use

2017-12-14 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64637/#review193873 --- Ship it! Ship It! - Dmytro Grinenko On Dec. 15, 2017, 3:03

Re: Review Request 64637: Livy/Livy2 Unable To Start Due to Address Already In Use

2017-12-14 Thread Jonathan Hurley
> On Dec. 14, 2017, 10:08 p.m., Dmytro Grinenko wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/livy_service.py > > Line 48 (original), 52 (patched) > > > > > > i don't

Re: Review Request 64637: Livy/Livy2 Unable To Start Due to Address Already In Use

2017-12-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64637/ --- (Updated Dec. 14, 2017, 10:38 p.m.) Review request for Ambari, Dmytro

Re: Review Request 64542: Kerberos support for OneFS

2017-12-14 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64542/#review193791 --- Ship it! Ship It! - Nate Cole On Dec. 14, 2017, 6:26 a.m.,

Re: Review Request 64606: Fix unit tests in feature branch to make them workable

2017-12-14 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64606/#review193792 --- Ship it! Ship It! - Dmytro Sen On Dec. 14, 2017, 12:23

Re: Review Request 64606: Fix unit tests in feature branch to make them workable

2017-12-14 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64606/ --- (Updated Гру. 14, 2017, 1:46 після полудня) Review request for Ambari, Attila

Re: Review Request 64594: AMBARI-22649. Library for querying clusterSettings and stackSettings for its contents in command*.json

2017-12-14 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64594/ --- (Updated Dec. 14, 2017, 9:02 a.m.) Review request for Ambari, Jayush Luniya

Review Request 64594: AMBARI-22649. Library for querying clusterSettings and stackSettings for its contents in command*.json

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

Re: Review Request 64542: Kerberos support for OneFS

2017-12-14 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64542/ --- (Updated Dec. 14, 2017, 11:26 a.m.) Review request for Ambari, Andrii