Re: Review Request 44724: AMBARI-12906: Alert notifications are created even if credential fields are left empty

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44724/ --- (Updated March 20, 2016, 5:50 a.m.) Review request for Ambari, Di Li and

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44972/ --- (Updated March 17, 2016, 6:11 p.m.) Review request for Ambari, Di Li and Sumit

Re: Review Request 44700: Ambari Views : each view should have separate log file for better troubleshooting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44700/#review124000 --- Ship it! Ship It! - DIPAYAN BHOWMICK On March 15, 2016,

Review Request 44923: Alerts: Allow alerts to emit a structured response

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44923/ --- Review request for Ambari, Dmytro Sen and Jonathan Hurley. Bugs: AMBARI-15445

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45012/#review124221 --- Fix it, then Ship it!

Re: Review Request 44953: Download client configurations does not work

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

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

> On Март 18, 2016, 3:29 п.п., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 135 > > > > > > Instead of 'special'

Re: Review Request 44887: Capacity Scheduler view is not working when using custom configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44887/#review123917 --- Ship it! Ship It! - DIPAYAN BHOWMICK On March 16, 2016,

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44972/#review124288 --- Hello Amruta, You listed the branch as 2.2.2. There is no

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44915/#review123883 --- ambari-web/app/messages.js (line 2586)

Re: Review Request 44868: Changed HAWQ directory descriptions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44868/ --- (Updated March 16, 2016, 10:42 p.m.) Review request for Ambari, Alejandro

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44983/ --- (Updated March 18, 2016, 10:43 a.m.) Review request for Ambari, Alejandro

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45054/#review124333 --- Ship it! Ship It! - Zhe (Joe) Wang On March 18, 2016, 9:46

Re: Review Request 44859: Cannot specify ZooKeeper heap size via UI

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44859/ --- (Updated Березень 17, 2016, 1:22 після полудня) Review request for Ambari,

Re: Review Request 44688: PXF should reach out to active namenode to get the delegation token on HDFS HA clusters

> On March 14, 2016, 5:10 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/PXF/3.0.0/package/alerts/api_status.py, > > line 140 > > > > > > Check for null (i.e. no active namenode) >

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44941/#review123980 --- Ship it! Ship It! - Sumit Mohanty On March 17, 2016, 1:09

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44958/ --- (Updated márc. 18, 2016, 8:39 de) Review request for Ambari, Laszlo Puskas,

Re: Review Request 44926: [DRAFT] Auto-retry on failure during RU/EU

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44926/#review124117 ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44957/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-15464

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44835/ --- (Updated March 18, 2016, 8:57 p.m.) Review request for Ambari, Alejandro

Re: Review Request 44878: Atlas Integration : Rename Atlas Configurations

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44878/#review123865 --- Ship it! Ship It! - Nate Cole On March 15, 2016, 8:57 p.m.,

Re: Review Request 44725: After exporting blueprint from ranger enabled cluster ranger.service.https.attrib.keystore.pass is exported

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44725/ --- (Updated March 17, 2016, 6:52 p.m.) Review request for Ambari, Di Li and

Re: Review Request 44565: Hive view: Visual Explain does not work when there are multiple statements in the query editor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44565/#review124065 --- Ship it! Ship It! - DIPAYAN BHOWMICK On March 17, 2016,

Re: Review Request 44926: [DRAFT] Auto-retry on failure during RU/EU

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44926/#review124202 ---

Re: Review Request 44923: Alerts: Allow alerts to emit a structured response

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44923/#review123914 --- Fix it, then Ship it!

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44987/#review124115 ---

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 HBas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44969/ --- Review request for Ambari, Jaimin Jetly and Sebastian Toader. Bugs:

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44916/#review123895 --- Shouldn't we also have a non-null check in the backend code? -

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

> On March 18, 2016, 5:44 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/python/ambari_server/serverConfiguration.py, line 230 > > > > > > This is doing rstrip instead, if the original string contains

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44915/ --- (Updated March 17, 2016, 12:47 a.m.) Review request for Ambari, Alexander

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44915/ --- (Updated March 17, 2016, 2:28 p.m.) Review request for Ambari, Alexander

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44916/ --- (Updated March 17, 2016, 1:48 p.m.) Review request for Ambari, Dmytro Sen,

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

> On March 16, 2016, 5:14 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorBlueprintProcessor.java, > > line 224 > > > > > > How can this

Re: Review Request 44962: file_system get_mount_point_for_dir works incorrect

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44962/#review124066 --- Ship it! Ship It! - Alejandro Fernandez On March 17, 2016,

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

> On March 16, 2016, 5:53 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py, > > line 106 > > > > > > This may not necessarily be a rolling

Re: Review Request 44725: After exporting blueprint from ranger enabled cluster ranger.service.https.attrib.keystore.pass is exported

> On March 14, 2016, 12:27 p.m., Di Li wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 2668 > > > > > > So any properties end with

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44956/#review124030 ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44915/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, jun aoki,

Re: Review Request 45056: Blueprint install using config_recommendation_strategy is not functional

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45056/#review124404 --- Ship it! Ship It! In the near future I will create a patch

Re: Review Request 44688: PXF should reach out to active namenode to get the delegation token on HDFS HA clusters

> On March 16, 2016, 8 p.m., Jayush Luniya wrote: > > Ship It! Thank you Jayush. No worries, will do merge now. - bhuvnesh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44688/#review123919

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

> On March 17, 2016, 8:53 a.m., Jonathan Hurley wrote: > > My only concern is for safety and clarify of code, does it makes sense to > > change the contract of the config to always return a non-null (if empty) > > map? I attempted to go this route, but I got stuck where the Jackson JSON

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45035/ --- Review request for Ambari, Alejandro Fernandez, Nate Cole, and Robert Levas.

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

> On March 16, 2016, 5:14 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorBlueprintProcessor.java, > > line 224 > > > > > > How can this

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45035/ --- (Updated March 18, 2016, 4:56 p.m.) Review request for Ambari, Alejandro

Re: Review Request 44773: Hive View : Upload Table : Tables are not deleted in case of exception.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44773/#review123976 --- Ship it! Ship It! - Pallav Kulshreshtha On March 14, 2016,

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44983/ --- Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. Bugs:

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44835/#review123892 --- Ship it!

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

> On March 16, 2016, 1:57 p.m., Sid Wagle wrote: > > Shouldn't we also have a non-null check in the backend code? I am not sure.. I guess it depends on the expectations. Historically it appears that the property map is expected to not be null. I am happy to add null protections in place I

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45030/#review124198 --- Ship it!

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

> On March 18, 2016, 5:08 p.m., jun aoki wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py, > > line 699 > > > > > > do you want to inline this since propertyvalue is used only

Re: Review Request 44880: AMBARI-15436 Supportability: Create 'Reset UI' button in Ambari's experimental page

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44880/#review124098 --- Ship it! Ship It! - Srimanth Gunturi On March 17, 2016,

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

> On March 18, 2016, 1:50 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java, > > line 453 > > > > > > I think adding a new column

Review Request 44926: [DRAFT] Auto-retry on failure during RU/EU

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44926/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-15446

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44956/#review124168 --- Ship it! Ship It! - Sebastian Toader On March 18, 2016,

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45054/ --- Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Yusaku Sako.

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44941/ --- (Updated March 17, 2016, 1:09 a.m.) Review request for Ambari, Aravindan

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44933/ --- Review request for Ambari, Alejandro Fernandez and Jayush Luniya. Bugs:

Re: Review Request 44959: Enable Ambari build process to automatically switch repo base urls in the repo files

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44959/#review124020 --- Ship it! Ship It! - Sumit Mohanty On March 17, 2016, 2:49

Re: Review Request 44880: AMBARI-15436 Supportability: Create 'Reset UI' button in Ambari's experimental page

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44880/ --- (Updated March 17, 2016, 8:52 p.m.) Review request for Ambari, Jaimin Jetly,

Re: Review Request 44923: Alerts: Allow alerts to emit a structured response

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44923/ --- (Updated Березень 17, 2016, 8:11 після полудня) Review request for Ambari,

Re: Review Request 44962: file_system get_mount_point_for_dir works incorrect

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44962/#review124045 --- Ship it! Ship It! - Dmytro Sen On Март 17, 2016, 4:03 п.п.,

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44931/ --- (Updated March 16, 2016, 6:08 p.m.) Review request for Ambari, Alejandro

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

> On márc. 17, 2016, 4:37 du, Robert Levas wrote: > > ambari-server/src/main/python/ambari_server_main.py, line 89 > > > > > > I am not familiar with these options, but it apears like no one really > > knows what

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44987/#review124306 ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44958/#review124172 --- Ship it! Ship It! - Robert Levas On March 18, 2016, 4:39

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

> On March 16, 2016, 12:24 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/setup_ranger_xml.py, > > lines 128-131 > > > > > > Do you need to: > >

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45012/ --- (Updated March 18, 2016, 7:34 p.m.) Review request for Ambari, Alejandro

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44956/ --- Review request for Ambari, Alejandro Fernandez, Laszlo Puskas, Oliver Szabo,

Re: Review Request 44687: Update RCO : PXF should start after HDFS

> On March 11, 2016, 9:06 a.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/role_command_order.json, > > line 14 > > > > > > RCO can be defined at service level > >

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45030/ --- Review request for Ambari, Jonathan Hurley, Robert Nettleton, and Sid Wagle.

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44972/ --- Review request for Ambari, Di Li and Sumit Mohanty. Bugs: AMBARI-15412

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45035/#review124232 ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44916/ --- Review request for Ambari, Dmytro Sen, Jonathan Hurley, Nate Cole, and Sid

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44931/ --- Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. Bugs:

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44958/#review124042 --- Fix it, then Ship it!

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

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

Re: Review Request 44868: Changed HAWQ directory descriptions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44868/#review123918 --- Ship it! Ship It! - Jayush Luniya On March 15, 2016, 10:51

Re: Review Request 44565: Hive view: Visual Explain does not work when there are multiple statements in the query editor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44565/ --- (Updated March 17, 2016, 4:07 p.m.) Review request for Ambari, DIPAYAN

Review Request 45041: Strange paths in ambari-server setup

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45041/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Dmytro Sen.

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44986/#review124222 --- Ship it! Ship It! - Nate Cole On March 17, 2016, 7 p.m.,

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44987/#review124128 ---

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

> On March 17, 2016, 11:10 p.m., Alexander Denissov wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/hawqsegment.py, > > line 45 > > > > > > plural, so perhaps "..they should

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44956/#review124068 --- Fix it, then Ship it!