Review Request 62239: AMBARI-21925 : Facilitate a parameter to provide HBase conf directory for atlas-env configurations.

2017-09-12 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62239/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya,

Re: Review Request 62125: AMBARI-21893 : NameNode Heap Usage (Daily) metric alert status flips to UNKNOWN intermittently when AMS HTTPS is enabled.

2017-09-12 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62125/#review185245 --- Fix it, then Ship it!

Re: Review Request 62250: AMBARI-21939 : Ambari HDFS Heatmaps are not showing data (Not Applicable) for bytes read, bytes written & DataNode Process Network I/O Utilization

2017-09-12 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62250/#review185246 --- Ship it! Ship It! - Sid Wagle On Sept. 12, 2017, 4:58 p.m.,

Re: Review Request 62250: AMBARI-21939 : Ambari HDFS Heatmaps are not showing data (Not Applicable) for bytes read, bytes written & DataNode Process Network I/O Utilization

2017-09-12 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62250/#review185250 --- Ship it! Ship It! - Jayush Luniya On Sept. 12, 2017, 4:58

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
> On Sept. 12, 2017, 1:07 p.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 145 (patched) > > > > > > while map function is present in

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Jonathan Hurley
> On Sept. 12, 2017, 9:07 a.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 125 (patched) > > > > > > According to code below, this

Re: Review Request 62235: Some clients are shown as installed for a hosts page even when they have not been installed through ambari

2017-09-12 Thread Ishan Bhatt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62235/#review185188 --- Ship it! Ship It! - Ishan Bhatt On Sept. 12, 2017, 4:24

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/#review185189 --- Ship it! Ship It! - Sandor Magyari On Sept. 12, 2017, 2:57

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/#review185190 --- Ship it! Ship It! - Attila Magyar On Sept. 12, 2017, 2:57

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/#review185191 --- Ship it! Ship It! - Robert Levas On Sept. 12, 2017, 10:57

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Ayub Pathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/#review185192 --- I dont see UTs added for BlueprintConfigurationProcessor.java.

Review Request 62249: Do not export YARN_CONF_DIR in 'yarn-env.sh' file

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

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/ --- (Updated Sept. 12, 2017, 5:18 p.m.) Review request for Ambari, Attila Magyar,

Review Request 62242: Implement breadcrumbs in Background Operations modal

2017-09-12 Thread Jason Golieb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62242/ --- Review request for Ambari, Jaimin Jetly and Yusaku Sako. Bugs: AMBARI-21933

Re: Review Request 62239: AMBARI-21925 : Facilitate a parameter to provide HBase conf directory for atlas-env configurations.

2017-09-12 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62239/#review185154 --- Ship it! Ship It! - Mugdha Varadkar On Sept. 12, 2017, 9:03

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62226/#review185174 --- Fix it, then Ship it! Ship It!

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62226/#review185170 --- Ship it! Ship It! - Dmitro Lisnichenko On Sept. 11, 2017,

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
> On Sept. 12, 2017, 1:07 p.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 125 (patched) > > > > > > According to code below, this

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Jonathan Hurley
> On Sept. 12, 2017, 9:07 a.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 125 (patched) > > > > > > According to code below, this

Re: Review Request 62125: AMBARI-21893 : NameNode Heap Usage (Daily) metric alert status flips to UNKNOWN intermittently when AMS HTTPS is enabled.

2017-09-12 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62125/ --- (Updated Sept. 12, 2017, 6:25 p.m.) Review request for Ambari, Dmytro Sen,

Re: Review Request 62249: Do not export YARN_CONF_DIR in 'yarn-env.sh' file

2017-09-12 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62249/#review185227 --- Ship it! Ship It! - Swapan Shridhar On Sept. 12, 2017, 4:47

Re: Review Request 62242: Implement breadcrumbs in Background Operations modal

2017-09-12 Thread Jason Golieb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62242/ --- (Updated Sept. 12, 2017, 7:07 p.m.) Review request for Ambari, Denys Buzhor,

Re: Review Request 62125: AMBARI-21893 : NameNode Heap Usage (Daily) metric alert status flips to UNKNOWN intermittently when AMS HTTPS is enabled.

2017-09-12 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62125/ --- (Updated Sept. 12, 2017, 7:21 p.m.) Review request for Ambari, Dmytro Sen,

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Ayub Pathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/#review185204 --- Ship it! - Ayub Pathan On Sept. 12, 2017, 3:18 p.m.,

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62226/#review185207 --- Ship it! Ship It! - Dmytro Grinenko On Sept. 12, 2017, 3:50

Re: Review Request 62235: Some clients are being additionally installed via Add service wizard

2017-09-12 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62235/ --- (Updated Sept. 12, 2017, 4:38 p.m.) Review request for Ambari, Ishan Bhatt,

Re: Review Request 62249: Do not export YARN_CONF_DIR in 'yarn-env.sh' file

2017-09-12 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62249/ --- (Updated Вер. 12, 2017, 4:47 після полудня) Review request for Ambari, Jayush

Review Request 62250: AMBARI-21939 : Ambari HDFS Heatmaps are not showing data (Not Applicable) for bytes read, bytes written & DataNode Process Network I/O Utilization

2017-09-12 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62250/ --- Review request for Ambari, Dmytro Sen, Jayush Luniya, and Sid Wagle. Bugs:

Re: Review Request 62249: Do not export YARN_CONF_DIR in 'yarn-env.sh' file

2017-09-12 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62249/#review185211 --- Ship it! Ship It! - Jayush Luniya On Sept. 12, 2017, 4:47

Re: Review Request 62125: AMBARI-21893 : NameNode Heap Usage (Daily) metric alert status flips to UNKNOWN intermittently when AMS HTTPS is enabled.

2017-09-12 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62125/ --- (Updated Sept. 12, 2017, 6:25 p.m.) Review request for Ambari, Dmytro Sen,

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Dmitro Lisnichenko
> On Sept. 12, 2017, 6:59 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java > > Lines 415-449 (patched) > > > > > > Does it make sense

Re: Review Request 62258: Stack selection page does not load the HDP stacks [Intermittent]

2017-09-12 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62258/#review185235 --- Fix it, then Ship it!

Review Request 62258: Stack selection page does not load the HDP stacks [Intermittent]

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

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Jonathan Hurley
> On Sept. 12, 2017, 11:59 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java > > Lines 415-449 (patched) > > > > > > Does it make sense

Re: Review Request 60358: Hivesevers's status shows issue for the thrift's port conflict, when hiveservers and spark thrift server installed in the same host.

2017-09-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60358/#review185187 --- What is the status of this review? - Nate Cole On June 22,

Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-21935

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62226/ --- (Updated Sept. 12, 2017, 10:21 a.m.) Review request for Ambari, Dmytro

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Jonathan Hurley
> On Sept. 12, 2017, 9:07 a.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 145 (patched) > > > > > > while map function is present in

Re: Review Request 61453: AMBARI-21634 Make sure Ambari also considers NIS users

2017-09-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61453/#review185186 --- This has been pushed. Please resolve your JIRA and close this

Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/ --- Review request for Ambari, Attila Magyar, Ayub Pathan, Robert Levas, and Sandor

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

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

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
> On Sept. 12, 2017, 3:42 p.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 126 (patched) > > > > > > if not supported_packages: > >

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/#review185203 ---

Re: Review Request 61898: Create Utility Script to support Solr Collection Data Retention/Purging/Archiving

2017-09-12 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61898/ --- (Updated Sept. 12, 2017, 3:30 p.m.) Review request for Ambari, Oliver Szabo

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62226/#review185197 --- Ship it! Ship It! - Dmytro Grinenko On Sept. 12, 2017, 2:21

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
> On Sept. 12, 2017, 1:07 p.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 145 (patched) > > > > > > while map function is present in

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Jonathan Hurley
> On Sept. 12, 2017, 11:42 a.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > > Lines 126 (patched) > > > > > > if not supported_packages: > >

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62226/ --- (Updated Sept. 12, 2017, 11:50 a.m.) Review request for Ambari, Dmytro

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62226/#review185206 --- ambari-server/src/test/python/stacks/utils/RMFTestCase.py Line

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-12 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/#review185205 --- Ship it! Ship It! - Robert Levas On Sept. 12, 2017, 11:18

Re: Review Request 62226: Use Correct Packages For Clients Where Stack Tools Support It

2017-09-12 Thread Jonathan Hurley
> On Sept. 12, 2017, 12:02 p.m., Dmytro Grinenko wrote: > > ambari-server/src/test/python/stacks/utils/RMFTestCase.py > > Line 142 (original), 142 (patched) > > > > > > i like this way more than that big "stairs"

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/#review185213 ---

Re: Review Request 62245: Auto fix enhancement to remove more than 1 selected configs

2017-09-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62245/#review185214 --- Ship it! Ship It! - Nate Cole On Sept. 12, 2017, 10:28

Re: Review Request 62250: AMBARI-21939 : Ambari HDFS Heatmaps are not showing data (Not Applicable) for bytes read, bytes written & DataNode Process Network I/O Utilization

2017-09-12 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62250/#review185215 ---

Re: Review Request 62250: AMBARI-21939 : Ambari HDFS Heatmaps are not showing data (Not Applicable) for bytes read, bytes written & DataNode Process Network I/O Utilization

2017-09-12 Thread Aravindan Vijayan
> On Sept. 12, 2017, 5:07 p.m., Dmytro Sen wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/widgets.json > > Line 340 (original), 340 (patched) > > > > > > Make sure to update the same for

Re: Review Request 62235: Some clients are being additionally installed via Add service wizard

2017-09-12 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62235/#review185220 --- Ship it! Ship It! - Yusaku Sako On Sept. 12, 2017, 4:38