Re: Review Request 44956: Topology host info is not cleared when a host is removed

2016-03-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44956/#review124068 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 45030: Replace 0.0.0.0 with hostname in ams-site/timeline.metrics.service.webapp.address

2016-03-18 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45030/#review124205 --- Ship it! Ship It! - Jonathan Hurley On March 18, 2016, 11:33

Re: Review Request 44886: AMBARI-15437 : Handle changing symlinks of ranger admin binaries for upgrade scenarios

2016-03-18 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44886/#review124008 --- Ship it! Ship It! - Jonathan Hurley On March 17, 2016, 6:13

Re: Review Request 45035: Restarting HDFS Before Upgrade Finalizing Does Not Supply the rollingUpgrade Flag

2016-03-18 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45035/#review124217 --- Ship it! ambari-server/src/main/java/org/apache/ambari/server/

Re: Review Request 44986: AMBARI-15474: Listen for changes to auto-start configuration and send them to the agent during heartbeats

2016-03-18 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44986/#review124220 --- ambari-server/src/main/java/org/apache/ambari/server/agent/HeartB

Re: Review Request 44960: AMBARI-15466. RBAC: Add dialog popup to explain roles

2016-03-18 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44960/#review124072 --- Ship it! Ship It! - Xi Wang On March 17, 2016, 7:54 a.m., Ri

Re: Review Request 44983: Service version display should be based on Version Definition

2016-03-18 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44983/#review124177 --- Ship it! Ship It! - Jonathan Hurley On March 17, 2016, 5:53

Re: Review Request 44957: [Ambari tarballs] ambari-server setup fails

2016-03-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44957/#review124016 --- Ship it! Ship It! - Vitalyi Brodetskyi On Березень 17, 2016,

Review Request 44941: Report AMS / Grafana Per Disk Metrics and other changes

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

Re: Review Request 44915: https://issues.apache.org/jira/browse/HAWQ-497

2016-03-18 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44915/ --- (Updated March 16, 2016, 9:46 p.m.) Review request for Ambari, Alexander Deniss

Re: Review Request 44958: Increase Ambari Server Perm gen default value

2016-03-18 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44958/#review124026 --- Ship it! Ship It! - Sebastian Toader On March 17, 2016, 3:57

Review Request 44987: Editing tmp dir from single to multiple dir doesn't trigger the backend change

2016-03-18 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44987/ --- Review request for Ambari. Bugs: AMBARI-15475 https://issues.apache.org/jir

Re: Review Request 45054: AMBARI-15443:Make Host bulk command menu item list stack driven instead of a hardcoded list in UI code

2016-03-18 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45054/#review124335 --- There are some assumptions about certain operations for certain sl

Re: Review Request 45030: Replace 0.0.0.0 with hostname in ams-site/timeline.metrics.service.webapp.address

2016-03-18 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45030/#review124190 --- Ship it! Ship It! - Robert Nettleton On March 18, 2016, 1:45

Re: Review Request 44931: Use Version Definition value for package-version when installing

2016-03-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44931/#review124064 --- Ship it! Ship It! - Alejandro Fernandez On March 17, 2016, 1

Re: Review Request 44931: Use Version Definition value for package-version when installing

2016-03-18 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44931/ --- (Updated March 17, 2016, 8:51 a.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 44956: Topology host info is not cleared when a host is removed

2016-03-18 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44956/ --- (Updated márc. 18, 2016, 9:27 de) Review request for Ambari, Alejandro Fernande

Re: Review Request 44886: AMBARI-15437 : Handle changing symlinks of ranger admin binaries for upgrade scenarios

2016-03-18 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44886/#review123860 --- ambari-server/src/main/resources/common-services/RANGER/0.4.0/pac

Re: Review Request 44835: AMBARI-15420: Refactor resource_management library

2016-03-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44835/#review124302 --- Ship it! Ship It! - Alejandro Fernandez On March 18, 2016, 8

Re: Review Request 45041: Strange paths in ambari-server setup

2016-03-18 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45041/#review124231 --- Ship it! Ship It! - Sumit Mohanty On March 18, 2016, 5:39 p.

Review Request 45056: Blueprint install using config_recommendation_strategy is not functional

2016-03-18 Thread Shantanu Mundkur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45056/ --- Review request for Ambari, Oliver Szabo and Robert Levas. Bugs: AMBARI-15454

Re: Review Request 44933: Apply the stack featurization prototype detailed on AMBARI-13364 to ZOOKEEPER service.

2016-03-18 Thread Juanjo Marron
> On March 18, 2016, 9:57 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.2/configuration/cluster-env.xml, > > line 24 > > > > > > Actually, this file must exist for the entire stac

Re: Review Request 44933: Apply the stack featurization prototype detailed on AMBARI-13364 to ZOOKEEPER service.

2016-03-18 Thread Juanjo Marron
> On March 18, 2016, 9:25 p.m., Jayush Luniya wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_fetaures.py, > > line 1 > > > > > > Minor, fix typo in file name OK.Thanks. I'll so

Re: Review Request 44958: Increase Ambari Server Perm gen default value

2016-03-18 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44958/#review124174 --- Ship it! Ship It! - Sandor Magyari On March 18, 2016, 8:39 a

Re: Review Request 44969: After upgrading from Ambari 2.1.2 to 2.2.1, HDP from 2.3.0 to 2.3.4 , user is getting 'Consistency Check Failed' error while trying to save configuration in Ranger, Hive and

2016-03-18 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44969/#review124156 --- ambari-server/src/main/resources/stacks/HDP/2.2/services/stack_ad

Re: Review Request 44956: Topology host info is not cleared when a host is removed

2016-03-18 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44956/#review124243 --- Ship it! Ship It! - Sandor Magyari On March 18, 2016, 9:27 a

Re: Review Request 44916: StackAdvisor needs to set config map for ams-grafana-env to meet non-null expectations

2016-03-18 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44916/#review123930 --- ambari-server/src/main/java/org/apache/ambari/server/api/services

Review Request 44962: file_system get_mount_point_for_dir works incorrect

2016-03-18 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44962/ --- Review request for Ambari and Dmytro Sen. Bugs: AMBARI-15468 https://issues

Re: Review Request 44969: After upgrading from Ambari 2.1.2 to 2.2.1, HDP from 2.3.0 to 2.3.4 , user is getting 'Consistency Check Failed' error while trying to save configuration in Ranger, Hive and

2016-03-18 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44969/ --- (Updated March 18, 2016, 5:18 p.m.) Review request for Ambari, Jaimin Jetly and

Re: Review Request 44933: Apply the stack featurization prototype detailed on AMBARI-13364 to ZOOKEEPER service.

2016-03-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44933/#review124298 --- ambari-server/src/main/resources/stacks/HDP/2.2/configuration/clu

Review Request 45069: HiveServerInteractive. Remove Password requirement from hive-interactive-site.xml.

2016-03-18 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45069/ --- Review request for Ambari, Jaimin Jetly, Sumit Mohanty, and Yusaku Sako. Bugs:

Re: Review Request 45054: AMBARI-15443:Make Host bulk command menu item list stack driven instead of a hardcoded list in UI code

2016-03-18 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45054/#review124303 --- Fix it, then Ship it! Thank you for adding this. Any unit tests

Re: Review Request 45069: HiveServerInteractive. Remove Password requirement from hive-interactive-site.xml.

2016-03-18 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45069/#review124354 --- Ship it! Ship It! - Yusaku Sako On March 19, 2016, 1:43 a.m.

Re: Review Request 44986: AMBARI-15474: Listen for changes to auto-start configuration and send them to the agent during heartbeats

2016-03-18 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44986/ --- (Updated March 18, 2016, 11:07 a.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 45012: Recommendation and validation logic for HAWQ hdfs-client.xml output.replace-datanode-on-failure property

2016-03-18 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45012/#review124226 --- Ship it! Ship It! - bhuvnesh chaudhary On March 18, 2016, 2:

Re: Review Request 44987: Editing tmp dir from single to multiple dir doesn't trigger the backend change

2016-03-18 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44987/ --- (Updated March 18, 2016, 10:17 p.m.) Review request for Ambari. Bugs: AMBARI-

Re: Review Request 44972: Improve error logging for install errors during blueprint deployments.

2016-03-18 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44972/#review124176 --- Ship it! Ship It! - Di Li On March 17, 2016, 6:11 p.m., Amru

Re: Review Request 44958: Increase Ambari Server Perm gen default value

2016-03-18 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44958/#review124175 --- Ship it! Ship It! - Oliver Szabo On March 18, 2016, 8:39 a.m