Review Request 57268: AMBARI-20287 - Filter in Customize Services Page doesn't bring up all properties that matches

2017-03-02 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57268/ --- Review request for Ambari, Jaimin Jetly and Yusaku Sako. Bugs: AMBARI-20287

Re: Review Request 57267: AMBARI-20247. Show full error while importing the workflow from encrypted file

2017-03-02 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57267/#review167796 --- Ship it! Ship It! - belliraj hb On March 3, 2017, 6:46 a.m.,

Review Request 57267: AMBARI-20247. Show full error while importing the workflow from encrypted file

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

Re: Review Request 57139: AMBARI-20208 : Atlas kafka servers should be configured using kafka listeners.

2017-03-02 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57139/#review167793 --- Ship it! Ship It! - Sriharsha Chintalapani On March 2, 2017,

Re: Review Request 57265: null in heapmap page

2017-03-02 Thread alice zhangxiaolu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57265/ --- (Updated 三月 3, 2017, 3:15 a.m.) Review request for Ambari, Dmytro Sen, Jaimin J

Review Request 57265: null in heapmap page

2017-03-02 Thread alice zhangxiaolu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57265/ --- Review request for Ambari, Dmytro Sen, Jaimin Jetly, Srimanth Gunturi, Sid Wagle

Review Request 57263: Fix values of ATS config params for HDP stack

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

Re: Review Request 57258: AMBARI-20279. HiveServerInteractive. Set value for "num_llap_nodes_for_llap_daemons" only if it exists in hive-interactive-site.

2017-03-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57258/#review167776 --- Ship it! Ship It! - Sumit Mohanty On March 2, 2017, 10:32 p.

Review Request 57259: AMBARI-20285. Spark2 Thrift Server stopped due to incompletely uploaded tar.gz

2017-03-02 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57259/ --- Review request for Ambari, Bikas Saha, Laszlo Puskas, Sandor Magyari, and Sebast

Re: Review Request 57258: AMBARI-20279. HiveServerInteractive. Set value for "num_llap_nodes_for_llap_daemons" only if it exists in hive-interactive-site.

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

Review Request 57258: AMBARI-20279. HiveServerInteractive. Set value for "num_llap_nodes_for_llap_daemons" only if it exists in hive-interactive-site.

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

Re: Review Request 57251: AMBARI-20276 : Perf - AMS scale test for 3000 node cluster

2017-03-02 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57251/#review167751 --- Ship it! Ship It! - Sid Wagle On March 2, 2017, 7:38 p.m., A

Re: Review Request 57250: NPE thrown if downscale is performed before all hosts registered with cluster.

2017-03-02 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57250/#review167746 --- Ship it! Ship It! - Attila Doroszlai On March 2, 2017, 8:31

Re: Review Request 57256: Porting changes made in 2.5/stack_advisor to 3.0/YARN/service_advisor for LLAP.

2017-03-02 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57256/ --- (Updated March 2, 2017, 9:43 p.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 57256: Porting changes made in 2.5/stack_advisor to 3.0/YARN/service_advisor for LLAP.

2017-03-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57256/#review167742 --- Ship it! Ship It! - Sumit Mohanty On March 2, 2017, 9:29 p.m

Re: Review Request 57251: AMBARI-20276 : Perf - AMS scale test for 3000 node cluster

2017-03-02 Thread Aravindan Vijayan
> On March 2, 2017, 8:18 p.m., Sid Wagle wrote: > > Can we consolidate the simualtor files and the ones used for split point > > calculation? The stack should be only source of truth. Created AMBARI-20283 to track the effort for the consolidation. - Aravindan ---

Re: Review Request 57255: Pre-Upgrade check message for disabling Auto Start is difficult to understand and take action on

2017-03-02 Thread Nate Cole
> On March 2, 2017, 4:04 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java > > Lines 322 (patched) > > > > > > "Turn the toggle switch off to Disabled

Re: Review Request 57256: Porting changes made in 2.5/stack_advisor to 3.0/YARN/service_advisor for LLAP.

2017-03-02 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57256/ --- (Updated March 2, 2017, 9:29 p.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 57256: Porting changes made in 2.5/stack_advisor to 3.0/YARN/service_advisor for LLAP.

2017-03-02 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57256/ --- (Updated March 2, 2017, 9:29 p.m.) Review request for Ambari, Alejandro Fernand

Review Request 57256: ADDENDUM. AMBARI-19827. HiveServer2 Interactive won't start in clusters with less memory. (changes for service_advisor).

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

Re: Review Request 57255: Pre-Upgrade check message for disabling Auto Start is difficult to understand and take action on

2017-03-02 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57255/#review167736 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Review Request 57255: Pre-Upgrade check message for disabling Auto Start is difficult to understand and take action on

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

Re: Review Request 57250: NPE thrown if downscale is performed before all hosts registered with cluster.

2017-03-02 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57250/#review167732 --- Ship it! Ship It! - Sandor Magyari On March 2, 2017, 7:31 p.

Re: Review Request 57253: Credential Store should be enabled by default on fresh installs

2017-03-02 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57253/#review167727 --- Ship it! Ship It! - Sumit Mohanty On March 2, 2017, 7:48 p.m

Re: Review Request 57251: AMBARI-20276 : Perf - AMS scale test for 3000 node cluster

2017-03-02 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57251/#review167726 --- Can we consolidate the simualtor files and the ones used for split

Re: Review Request 57250: NPE thrown if downscale is performed before all hosts registered with cluster.

2017-03-02 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57250/#review167723 --- Ship it! Ship It! - Robert Nettleton On March 2, 2017, 7:31

Re: Review Request 57253: Credential Store should be enabled by default on fresh installs

2017-03-02 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57253/ --- (Updated March 2, 2017, 7:48 p.m.) Review request for Ambari, Alejandro Fernand

Review Request 57253: Credential Store should be enabled by default on fresh installs

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

Review Request 57251: AMBARI-20276 : Perf - AMS scale test for 3000 node cluster

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

Re: Review Request 57250: NPE thrown if downscale is performed before all hosts registered with cluster.

2017-03-02 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57250/#review167720 --- Ship it! Ship It! - Robert Levas On March 2, 2017, 2:31 p.m.

Review Request 57250: NPE thrown if downscale is performed before all hosts registered with cluster.

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

Re: Review Request 57139: AMBARI-20208 : Atlas kafka servers should be configured using kafka listeners.

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

Re: Review Request 57147: Added support for processing custom command script timeout

2017-03-02 Thread Alejandro Fernandez
> On March 1, 2017, 7:07 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml > > Lines 64 (patched) > > > > > > I'm still opposed to this patch fundamental

Re: Review Request 57248: Visual explain is not routing to visual explain tab

2017-03-02 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57248/#review167699 --- Ship it! Ship It! - Abhishek Kumar On March 2, 2017, 6:08 p.

Re: Review Request 57248: Visual explain is not routing to visual explain tab

2017-03-02 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57248/#review167697 --- Ship it! Ship It! - DIPAYAN BHOWMICK On March 2, 2017, 6:08

Review Request 57248: Visual explain is not routing to visual explain tab

2017-03-02 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57248/ --- Review request for Ambari, Abhishek Kumar, DIPAYAN BHOWMICK, Gaurav Nagar, Nitir

Re: Review Request 57247: Create Widget: Metrics select is empty after refresh

2017-03-02 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57247/#review167693 --- Ship it! Ship It! - Aleksandr Kovalenko On Март 2, 2017, 5:4

Review Request 57247: Create Widget: Metrics select is empty after refresh

2017-03-02 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57247/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-20273 https

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-02 Thread Anita Jebaraj
> On March 1, 2017, 4:28 p.m., Di Li wrote: > > ambari-server/conf/unix/ambari-blacklist.properties > > Lines 1 (patched) > > > > > > I am not sure Ambari should ship with empty files. > > > > I would suggest

Re: Review Request 57245: Call to fetch Quick Links host fails

2017-03-02 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57245/#review167689 --- Ship it! Ship It! - Aleksandr Kovalenko On Март 2, 2017, 4:3

Review Request 57245: Call to fetch Quick Links host fails

2017-03-02 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57245/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-20271 https

Re: Review Request 57204: After regenerate keytabs post Ambari upgrade yarn.nodemanager.linux-container-executor.cgroups.mount-path property got added with blank value

2017-03-02 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57204/#review167688 --- Ship it! Ship It! - Robert Levas On March 2, 2017, 7:53 a.m.

Re: Review Request 57215: Startup Annotation Scanning Takes Too Long

2017-03-02 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57215/ --- (Updated March 2, 2017, 10:48 a.m.) Review request for Ambari, Nate Cole and Ro

Re: Review Request 57215: Startup Annotation Scanning Takes Too Long

2017-03-02 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57215/#review167677 --- Ship it! Ship It! - Nate Cole On March 1, 2017, 3:51 p.m., J

Re: Review Request 57238: Ambari shouldn't remove secure ACLs from /zkdtsm_

2017-03-02 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57238/#review167675 --- Ship it! Ship It! - Robert Levas On March 2, 2017, 7:39 a.m.

Re: Review Request 57240: AMBARI-20268.WFM generates duplicate kill nodes in XML if decision paths have multiple transition to kill node.

2017-03-02 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57240/#review167674 --- Ship it! Ship It! - Gaurav Nagar On March 2, 2017, 10:22 a.m

Re: Review Request 57237: AMBARI-20266.Duplicate node error is shown if kill node transition exists from decision node in WFM

2017-03-02 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57237/#review167673 --- Ship it! Ship It! - Gaurav Nagar On March 2, 2017, 9:11 a.m.

Re: Review Request 57238: Ambari shouldn't remove secure ACLs from /zkdtsm_

2017-03-02 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57238/#review167672 --- Ship it! Ship It! - Sebastian Toader On March 2, 2017, 1:39

Re: Review Request 57204: After regenerate keytabs post Ambari upgrade yarn.nodemanager.linux-container-executor.cgroups.mount-path property got added with blank value

2017-03-02 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57204/#review167671 --- Ship it! Ship It! - Dmitro Lisnichenko On March 2, 2017, 2:5

Re: Review Request 57204: After regenerate keytabs post Ambari upgrade yarn.nodemanager.linux-container-executor.cgroups.mount-path property got added with blank value

2017-03-02 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57204/#review167670 --- Ship it! Ship It! - Vitalyi Brodetskyi On Березень 2, 2017,

Re: Review Request 57204: After regenerate keytabs post Ambari upgrade yarn.nodemanager.linux-container-executor.cgroups.mount-path property got added with blank value

2017-03-02 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57204/ --- (Updated March 2, 2017, 12:53 p.m.) Review request for Ambari, Dmitro Lisnichen

Review Request 57238: Ambari shouldn't remove secure ACLs from /zkdtsm_

2017-03-02 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57238/ --- Review request for Ambari, Attila Doroszlai, Laszlo Puskas, Robert Levas, and Se

Re: Review Request 57139: AMBARI-20208 : Atlas kafka servers should be configured using kafka listeners.

2017-03-02 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57139/ --- (Updated March 2, 2017, 10:52 a.m.) Review request for Ambari, Alejandro Fernan

Review Request 57240: AMBARI-20268.WFM generates duplicate kill nodes in XML if decision paths have multiple transition to kill node.

2017-03-02 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57240/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 57240: AMBARI-20268.WFM generates duplicate kill nodes in XML if decision paths have multiple transition to kill node.

2017-03-02 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57240/#review167665 --- Ship it! Ship It! - belliraj hb On March 2, 2017, 10:22 a.m.

Re: Review Request 57100: Select operators should not appear before fetch

2017-03-02 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57100/#review167658 --- Ship it! Ship It! - Abhishek Kumar On Feb. 27, 2017, 7:29 p.

Re: Review Request 57237: AMBARI-20266.Duplicate node error is shown if kill node transition exists from decision node in WFM

2017-03-02 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57237/#review167657 --- Ship it! Ship It! - belliraj hb On March 2, 2017, 9:11 a.m.,

Review Request 57237: AMBARI-20266.Duplicate node error is shown if kill node transition exists from decision node in WFM

2017-03-02 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57237/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 57147: Added support for processing custom command script timeout

2017-03-02 Thread Laszlo Puskas
> On March 1, 2017, 7:07 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml > > Lines 64 (patched) > > > > > > I'm still opposed to this patch fundamental