Re: Review Request 56199: HDP 3.0 TP - Support changed configs and scripts for HDFS

2017-02-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56199/#review164086 --- Ship it! Ship It! - Sumit Mohanty On Feb. 3, 2017, 2:26

Re: Review Request 56261: Perf: start/stop all actions works much slower after few days of testing

2017-02-02 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56261/#review164083 --- Ship it! Ship It! - Sid Wagle On Feb. 3, 2017, 12:24 a.m.,

Re: Review Request 56266: The user must be clearly communicated about YARN pre-emption requirements when Hive LLAP is enabled

2017-02-02 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56266/ --- (Updated Feb. 3, 2017, 3:44 a.m.) Review request for Ambari, Jaimin Jetly and

Re: Review Request 56266: The user must be clearly communicated about YARN pre-emption requirements when Hive LLAP is enabled

2017-02-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56266/#review164079 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 3, 2017,

Review Request 56264: Add "live_hosts" metric in AMS for apps

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

Review Request 56266: The user must be clearly communicated about YARN pre-emption requirements when Hive LLAP is enabled

2017-02-02 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56266/ --- Review request for Ambari, Jaimin Jetly and Yusaku Sako. Bugs: AMBARI-19859

Re: Review Request 56199: HDP 3.0 TP - Support changed configs and scripts for HDFS

2017-02-02 Thread Alejandro Fernandez
> On Feb. 3, 2017, 2:31 a.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/stacks/HDP/2.2/services/HIVE/metainfo.xml, > > line 20 > > > > > > Why was this change needed? Is it just to make it explicit?

Re: Review Request 56199: HDP 3.0 TP - Support changed configs and scripts for HDFS

2017-02-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56199/#review164072 ---

Re: Review Request 56199: HDP 3.0 TP - Support changed configs and scripts for HDFS

2017-02-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56199/#review164071 --- ambari-server/src/test/python/unitTests.py

Re: Review Request 56199: HDP 3.0 TP - Support changed configs and scripts for HDFS

2017-02-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56199/ --- (Updated Feb. 3, 2017, 2:26 a.m.) Review request for Ambari, Dmytro Sen, Sid

Re: Review Request 56261: Perf: start/stop all actions works much slower after few days of testing

2017-02-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56261/#review164064 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 3, 2017,

Review Request 56261: Perf: start/stop all actions works much slower after few days of testing

2017-02-02 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56261/ --- Review request for Ambari, Alejandro Fernandez, Myroslav Papirkovskyy, and Sid

Re: Review Request 56230: Stack advisor issues encountered

2017-02-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56230/#review164039 ---

Re: Review Request 56230: Stack advisor issues encountered

2017-02-02 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56230/#review164034 --- Ship it! Ship It! - Sid Wagle On Feb. 2, 2017, 6:08 p.m.,

Re: Review Request 56232: AMBARI-19810. Remove upgrade logic in UpdateCatalog250 for tez-interactive-site's 'tez.runtime.io.sort.mb' and 'tez.runtime.unordered.output.buffer.size-mb'.

2017-02-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56232/#review164036 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 2, 2017,

Review Request 56232: AMBARI-19810. Remove upgrade logic in UpdateCatalog250 for tez-interactive-site's 'tez.runtime.io.sort.mb' and 'tez.runtime.unordered.output.buffer.size-mb'.

2017-02-02 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56232/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 53686: Stage and Request status should be persisted in the database

2017-02-02 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53686/#review164025 --- Have you tried this patch on a large cluster with success?

Re: Review Request 56059: Preview: Package Installation fails due to error in Berkeley DB library

2017-02-02 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56059/#review164024 --- Ship it! Ship It! - Jonathan Hurley On Feb. 2, 2017, 11:16

Re: Review Request 56059: Preview: Package Installation fails due to error in Berkeley DB library

2017-02-02 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56059/#review164021 --- Ship it! Ship It! - Nate Cole On Feb. 2, 2017, 11:16 a.m.,

Re: Review Request 56235: findLatestServiceConfigsByStack query returns deleted config group

2017-02-02 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56235/#review164018 --- Ship it! Ship It! - Nate Cole On Feb. 2, 2017, 3:01 p.m.,

Re: Review Request 56233: Increase default timeout and threadpool size for the external script to work on slower machines

2017-02-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56233/#review164014 --- Ship it! Ship It! - Sumit Mohanty On Feb. 2, 2017, 7:50

Review Request 56235: findLatestServiceConfigsByStack query returns deleted config group

2017-02-02 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56235/ --- Review request for Ambari, Di Li and Nate Cole. Bugs: AMBARI-19813

Re: Review Request 56224: Improve Log Feeder simulation to help scale testing

2017-02-02 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56224/#review164012 --- Ship it! Ship It! - Robert Nettleton On Feb. 2, 2017, 1

Re: Review Request 56179: Add infra-solr-plugin for authorization (with Kerberos)

2017-02-02 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56179/#review164011 --- Ship it! Ship It! - Robert Nettleton On Feb. 2, 2017, 4:23

Review Request 56233: Increase default timeout and threadpool size for the external script to work on slower machines

2017-02-02 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56233/ --- Review request for Ambari, Sid Wagle and Yusaku Sako. Bugs: AMBARI-19433

Re: Review Request 55099: Disable auto start before RU/EU and enable during finalization phase

2017-02-02 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55099/ --- (Updated Feb. 2, 2017, 7:15 p.m.) Review request for Ambari and Aleksandr

Review Request 56231: AMBARI-19829: Several HDFS/YARN widgets on Heatmaps show N/A

2017-02-02 Thread Qin Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56231/ --- Review request for Ambari. Bugs: AMBARI-19829

Re: Review Request 56059: Preview: Package Installation fails due to error in Berkeley DB library

2017-02-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56059/#review164007 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 2, 2017,

Re: Review Request 56196: UI changes to resolve discrepancies between what the stack vs Ambari reports as "live" for NodeManagers

2017-02-02 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56196/#review164005 --- Ship it! Ship It! - Yusaku Sako On Feb. 1, 2017, 11:56

Review Request 56230: Stack advisor issues encountered

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

Re: Review Request 56227: ambari-server start failed with exit code 1.

2017-02-02 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56227/#review164001 --- Ship it! Ship It! - Jayush Luniya On Feb. 2, 2017, 2:19

Re: Review Request 56224: Improve Log Feeder simulation to help scale testing

2017-02-02 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56224/#review164000 --- Ship it! Ship It! - Oliver Szabo On Feb. 2, 2017, 1 p.m.,

Re: Review Request 56228: Recent workflows in Workflow designer should be in descending order of time of updation

2017-02-02 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56228/#review163999 --- Ship it! Ship It! - Nitiraj Rathore On Feb. 2, 2017, 3:10

Re: Review Request 56141: Add precheck for Auto-Start being disabled

2017-02-02 Thread Jonathan Hurley
> On Feb. 1, 2017, 1:08 p.m., Sandor Magyari wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/AutoStartDisabledCheck.java, > > line 67 > > > > > > ["AUTO_INSTALL_START", "AUTO_START", "FULL"] >

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Dmitro Lisnichenko
> On Feb. 2, 2017, 4:19 p.m., Dmitro Lisnichenko wrote: > > ambari-agent/src/test/python/ambari_agent/TestCustomServiceOrchestrator.py, > > line 324 > > > > > > I think this assertion makes no sence since we would

Re: Review Request 56229: Workflow name mandatory for save in Coordinator and Bundle

2017-02-02 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56229/#review163994 --- Ship it! Ship It! - belliraj hb On Feb. 2, 2017, 4:37 p.m.,

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Attila Doroszlai
> On Feb. 2, 2017, 3:19 p.m., Dmitro Lisnichenko wrote: > > ambari-agent/src/test/python/ambari_agent/TestCustomServiceOrchestrator.py, > > line 324 > > > > > > I think this assertion makes no sence since we would

Review Request 56229: Workflow name mandatory for save in Coordinator and Bundle

2017-02-02 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56229/ --- Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj

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

2017-02-02 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56020/#review163987 --- Ship it! Ship It! - Sid Wagle On Feb. 2, 2017, 4:03 p.m.,

Re: Review Request 56179: Add infra-solr-plugin for authorization (with Kerberos)

2017-02-02 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56179/ --- (Updated Feb. 2, 2017, 4:23 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 53686: Stage and Request status should be persisted in the database

2017-02-02 Thread Jonathan Hurley
> On Dec. 15, 2016, 11:49 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CalculatedStatus.java, > > lines 282-287 > > > > > > Would this count states like

Re: Review Request 56059: Preview: Package Installation fails due to error in Berkeley DB library

2017-02-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56059/ --- (Updated Feb. 2, 2017, 6:16 p.m.) Review request for Ambari, Alejandro

Re: Review Request 56059: Preview: Package Installation fails due to error in Berkeley DB library

2017-02-02 Thread Dmitro Lisnichenko
> On Feb. 1, 2017, 11:40 p.m., Nate Cole wrote: > > ambari-common/src/main/python/ambari_commons/shell.py, line 42 > > > > > > Is it possible that these paths are different between redhat6 and > > redhat7 (or any of

Re: Review Request 56141: Add precheck for Auto-Start being disabled

2017-02-02 Thread Nate Cole
> On Feb. 1, 2017, 1:43 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/AutoStartDisabledCheck.java, > > line 67 > > > > > > To make this robust against future

Re: Review Request 56141: Add precheck for Auto-Start being disabled

2017-02-02 Thread Nate Cole
> On Feb. 1, 2017, 1:08 p.m., Sandor Magyari wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/AutoStartDisabledCheck.java, > > line 67 > > > > > > ["AUTO_INSTALL_START", "AUTO_START", "FULL"] >

Re: Review Request 56141: Add precheck for Auto-Start being disabled

2017-02-02 Thread Nate Cole
> On Feb. 1, 2017, 1:08 p.m., Sandor Magyari wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/AutoStartDisabledCheck.java, > > line 67 > > > > > > ["AUTO_INSTALL_START", "AUTO_START", "FULL"] >

Re: Review Request 56141: Add precheck for Auto-Start being disabled

2017-02-02 Thread Jonathan Hurley
> On Feb. 1, 2017, 1:08 p.m., Sandor Magyari wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/AutoStartDisabledCheck.java, > > line 67 > > > > > > ["AUTO_INSTALL_START", "AUTO_START", "FULL"] >

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

2017-02-02 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56020/ --- (Updated Фев. 2, 2017, 4:03 п.п.) Review request for Ambari, Aravindan

Re: Review Request 56179: Add infra-solr-plugin for authorization (with Kerberos)

2017-02-02 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56179/ --- (Updated Feb. 2, 2017, 4:02 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 56228: Recent workflows in Workflow designer should be in descending order of time of updation

2017-02-02 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56228/#review163977 --- Ship it! Ship It! - belliraj hb On Feb. 2, 2017, 3:10 p.m.,

Review Request 56228: Recent workflows in Workflow designer should be in descending order of time of updation

2017-02-02 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56228/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/#review163971 ---

Review Request 56227: ambari-server start failed with exit code 1.

2017-02-02 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56227/ --- Review request for Ambari and Jayush Luniya. Bugs: AMBARI-19851

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/#review163969 --- Ship it! Ship It! - Sebastian Toader On Feb. 2, 2017, 2:36

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Attila Doroszlai
> On Feb. 2, 2017, 2:21 p.m., Andrew Onischuk wrote: > > ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py, > > line 320 > > > > > > Hi Attila. > > > > Checking if the key in

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/#review163968 --- Ship it! Ship It! - Andrew Onischuk On Feb. 2, 2017, 1:36

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/ --- (Updated Feb. 2, 2017, 2:36 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/#review163966 ---

Review Request 56225: AMBARI-19846. ambari-agent.out filled with KeyError

2017-02-02 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56225/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Sandor Magyari,

Review Request 56224: Improve Log Feeder simulation to help scale testing

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

Re: Review Request 56179: Add infra-solr-plugin for authorization (with Kerberos)

2017-02-02 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56179/#review163965 --- In my past experience if a service runs on multiple hosts in

Re: Review Request 56179: Add infra-solr-plugin for authorization (with Kerberos)

2017-02-02 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56179/#review163963 --- Ship it! Ship It! - Miklos Gergely On Feb. 1, 2017, 8:46

Re: Review Request 56205: AMBARI-19842:Custom Action should be created without prompting for the action type

2017-02-02 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56205/#review163961 --- Ship it! Ship It! - Gaurav Nagar On Feb. 2, 2017, 7:01

Re: Review Request 56207: AMBARI-19843. Publish asset has issues when different users logins

2017-02-02 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56207/#review163962 --- Ship it! Ship It! - Gaurav Nagar On Feb. 2, 2017, 9:30

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

2017-02-02 Thread Attila Magyar
> On Jan. 31, 2017, 6:53 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.6/services/YARN/configuration/yarn-env.xml, > > line 21 > > > > > > Does this file differ from the one used

Re: Review Request 56207: AMBARI-19843. Publish asset has issues when different users logins

2017-02-02 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56207/#review163948 --- Ship it! Ship It! - belliraj hb On Feb. 2, 2017, 9:30 a.m.,

Review Request 56207: AMBARI-19843. Publish asset has issues when different users logins

2017-02-02 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56207/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and