Re: Review Request 46998: AMBARI-16264 Clean up local DB when get back to step 0 of installer wizard

2016-05-04 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46998/#review131780 --- Ship it! Ship It! - Jaimin Jetly On May 5, 2016, 12:44

Review Request 46998: AMBARI-16264 Clean up local DB when get back to step 0 of installer wizard

2016-05-04 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46998/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Srimanth Gunturi, Xi

Re: Review Request 46927: AMBARI-16210. Update Storm Ambari stack definition to work with 1.0 release.

2016-05-04 Thread Sriharsha Chintalapani
> On May 5, 2016, 12:26 a.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/pacemaker.py, > > line 40 > > > > > > Shouldnt return "storm-client" thats

Re: Review Request 46927: AMBARI-16210. Update Storm Ambari stack definition to work with 1.0 release.

2016-05-04 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46927/ --- (Updated May 5, 2016, 12:32 a.m.) Review request for Ambari, Jaimin Jetly and

Re: Review Request 46996: HAWQ service can be stopped and checked even when STANDBY MASTER is stopped

2016-05-04 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46996/#review131771 --- Ship it! Ship It! - bhuvnesh chaudhary On May 4, 2016,

Re: Review Request 46996: HAWQ service can be stopped and checked even when STANDBY MASTER is stopped

2016-05-04 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46996/#review131769 --- Ship it! ambari-web/app/controllers/main/service/item.js

Re: Review Request 46976: AMBARI-16250: Create STORM service versions in common-services

2016-05-04 Thread Jayush Luniya
> On May 4, 2016, 11:37 p.m., Sriharsha Chintalapani wrote: > > Lets wait till AMBARI-16210 gets merged in. Since we are shipping 1.0 with > > Ambari 2.4 that needs to be part of this patch as well. > > Jayush Luniya wrote: > @Sriharsha, > Sumit already committed that patch and the

Re: Review Request 46927: AMBARI-16210. Update Storm Ambari stack definition to work with 1.0 release.

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46927/#review131767 ---

Re: Review Request 46996: HAWQ service can be stopped and checked even when STANDBY MASTER is stopped

2016-05-04 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46996/#review131764 --- Ship it! Ship It! - Lav Jain On May 4, 2016, 11:52 p.m.,

Re: Review Request 46996: HAWQ service can be stopped and checked even when STANDBY MASTER is stopped

2016-05-04 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46996/#review131762 --- Ship it! Ship It! - Matt On May 4, 2016, 4:52 p.m.,

Review Request 46996: HAWQ service can be stopped and checked even when STANDBY MASTER is stopped

2016-05-04 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46996/ --- Review request for Ambari, bhuvnesh chaudhary, Goutam Tadi, jun aoki, Lav Jain,

Re: Review Request 46976: AMBARI-16250: Create STORM service versions in common-services

2016-05-04 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46976/#review131759 --- Lets wait till AMBARI-16210 gets merged in. Since we are shipping

Re: Review Request 46949: PXF alert: change the message to make it more meaningful when both namenodes are down on Secured HA cluster.

2016-05-04 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46949/#review131752 --- Ship it! Ship It! - Alexander Denissov On May 4, 2016, 9:28

Re: Review Request 46987: Show the connect string with the HiveServer instance

2016-05-04 Thread Zhe (Joe) Wang
> On May 4, 2016, 8:04 p.m., Zhe (Joe) Wang wrote: > > The width of the field for this string seems be too small. I can only see > > "jdbc:hive2://c6402". > > Jaimin Jetly wrote: > Thats intentional as the string can be long depending upon zookeeper > quorom value. so it is ellipses

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-05-04 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/#review131747 --- Ship it! We have tested the following changes: - The logic for

Re: Review Request 46949: PXF alert: change the message to make it more meaningful when both namenodes are down on Secured HA cluster.

2016-05-04 Thread jun aoki
> On May 4, 2016, 4:45 p.m., Alexander Denissov wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/namenode_ha_utils.py, > > line 143 > > > > > > The code in line 57 is: > > > >

Re: Review Request 46949: PXF alert: change the message to make it more meaningful when both namenodes are down on Secured HA cluster.

2016-05-04 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46949/ --- (Updated May 4, 2016, 9:28 p.m.) Review request for Ambari. Bugs:

Re: Review Request 46944: AMBARI-16230: Sporadic errors when deploying the cluster

2016-05-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46944/#review131744 --- Ship it! Ship It! - Jonathan Hurley On May 4, 2016, 5:09

Re: Review Request 46971: AMBARI-16248: Create KAFKA service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46971/ --- (Updated May 4, 2016, 9:16 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 46981: Logsearch: Basic stack advisor recommendation (shard/replication)

2016-05-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46981/#review131740 --- Ship it! Ship It! - Miklos Gergely On May 4, 2016, 8:49

Re: Review Request 46947: AMBARI-16235: Add beeline, hive-exec and hive log properties to /etc/hive2/conf

2016-05-04 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46947/#review131736 --- Ship it! Ship It! - Sumit Mohanty On May 4, 2016, 7:21

Re: Review Request 46987: Show the connect string with the HiveServer instance

2016-05-04 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46987/#review131733 --- The width of the field for this string seems be too small. I can

Re: Review Request 46987: Show the connect string with the HiveServer instance

2016-05-04 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46987/#review131734 --- Ship it! Ship It! - Zhe (Joe) Wang On May 4, 2016, 5:52

Re: Review Request 46947: AMBARI-16235: Add beeline, hive-exec and hive log properties to /etc/hive2/conf

2016-05-04 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46947/ --- (Updated May 4, 2016, 12:21 p.m.) Review request for Ambari, Sumit Mohanty and

Re: Review Request 46992: AMBARI-16259 Retry of cluster install does not work

2016-05-04 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46992/#review131731 --- Ship it! Ship It! - Jaimin Jetly On May 4, 2016, 6:42 p.m.,

Review Request 46992: AMBARI-16259 Retry of cluster install does not work

2016-05-04 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46992/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Srimanth Gunturi, Xi

Re: Review Request 46987: Show the connect string with the HiveServer instance

2016-05-04 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46987/#review131728 --- Ship it! Ship It! - Richard Zang On May 4, 2016, 5:52 p.m.,

Re: Review Request 46984: RBAC: Clean up roles after ambari upgrade

2016-05-04 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46984/#review131727 --- Ship it! Ship It! - Sebastian Toader On May 4, 2016, 6:49

Re: Review Request 46991: Do not let Stack upgrade start if there Ambari DB has inconsistencies

2016-05-04 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46991/#review131723 --- Ship it! Ship It! - Dmytro Sen On Май 4, 2016, 6:17 п.п.,

Review Request 46991: Do not let Stack upgrade start if there Ambari DB has inconsistencies

2016-05-04 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46991/ --- Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, and Sumit Mohanty.

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-05-04 Thread Matt
> On May 3, 2016, 4:09 p.m., Matt wrote: > > The functionality of ServiceAdvisor works as expected. However more > > clarification is needed on how to use inheritance and override service > > advisor defined in common-services. > > For example: > > HAWQ200ServiceAdvisor is defined under

Re: Review Request 46951: AMBARI-16238 VDF UI edits: #1

2016-05-04 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46951/#review131717 --- Ship it! Ship It! - Jaimin Jetly On May 4, 2016, 12:05

Re: Review Request 46984: RBAC: Clean up roles after ambari upgrade

2016-05-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46984/#review131716 --- Ship it! Ship It! - Nate Cole On May 4, 2016, 12:49 p.m.,

Review Request 46987: Show the connect string with the HiveServer instance

2016-05-04 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46987/ --- Review request for Ambari, Richard Zang and Srimanth Gunturi. Bugs:

Review Request 46984: RBAC: Clean up roles after ambari upgrade

2016-05-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46984/ --- Review request for Ambari, Jonathan Hurley, Myroslav Papirkovskyy, Nate Cole,

Re: Review Request 46949: PXF alert: change the message to make it more meaningful when both namenodes are down on Secured HA cluster.

2016-05-04 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46949/#review131705 ---

Re: Review Request 46981: Logsearch: Basic stack advisor recommendation (shard/replication)

2016-05-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46981/#review131704 --- The first 5 rows of the function are also unnecessary if the

Re: Review Request 46752: Ambari Web Service installation does not comply with Final Attribute. (isFinal is skipped)

2016-05-04 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46752/ --- (Updated May 4, 2016, 4:30 p.m.) Review request for Ambari, Andriy Babiichuk,

Re: Review Request 46981: Logsearch: Basic stack advisor recommendation (shard/replication)

2016-05-04 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46981/#review131698 --- Ship it! Ship It! - Robert Nettleton On May 4, 2016, 3:30

Re: Review Request 46973: Amanded handling of excluded config-types handling in case of blueprint deployments

2016-05-04 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46973/#review131696 --- Ship it! Ship It! - Robert Nettleton On May 4, 2016, 3:11

Re: Review Request 46941: Add Zookeeper dependency to LogSearch components for better Blueprint validation

2016-05-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46941/#review131695 --- Ship it! Ship It! - Miklos Gergely On May 3, 2016, 4:42

Re: Review Request 46973: Amanded handling of excluded config-types handling in case of blueprint deployments

2016-05-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46973/#review131694 --- Ship it! Ship It! - Robert Levas On May 4, 2016, 11:11

Re: Review Request 46973: Amanded handling of excluded config-types handling in case of blueprint deployments

2016-05-04 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46973/#review131689 --- Ship it! Ship It! - Sandor Magyari On May 4, 2016, 3:11

Re: Review Request 46973: Amanded handling of excluded config-types handling in case of blueprint deployments

2016-05-04 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46973/ --- (Updated May 4, 2016, 3:11 p.m.) Review request for Ambari, Robert Levas,

Re: Review Request 46944: AMBARI-16230: Sporadic errors when deploying the cluster

2016-05-04 Thread Jonathan Hurley
> On May 4, 2016, 10:07 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ServiceResourceProvider.java, > > lines 360-361 > > > > > > Might want to try/catch

Re: Review Request 46977: Hive metastore alerts are present after upgrade/downgrade and enabling security

2016-05-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46977/#review131677 --- Ship it! Ship It! - Jonathan Hurley On May 4, 2016, 10:11

Re: Review Request 46976: AMBARI-16250: Create STORM service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46976/ --- (Updated May 4, 2016, 2:21 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 46976: AMBARI-16250: Create STORM service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46976/ --- (Updated May 4, 2016, 2:19 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-05-04 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/ --- (Updated May 4, 2016, 2:17 p.m.) Review request for Ambari, Alexander

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-05-04 Thread Tim Thorpe
> On May 3, 2016, 11:09 p.m., Matt wrote: > > The functionality of ServiceAdvisor works as expected. However more > > clarification is needed on how to use inheritance and override service > > advisor defined in common-services. > > For example: > > HAWQ200ServiceAdvisor is defined under

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-05-04 Thread Tim Thorpe
> On May 3, 2016, 11:09 p.m., Matt wrote: > > The functionality of ServiceAdvisor works as expected. However more > > clarification is needed on how to use inheritance and override service > > advisor defined in common-services. > > For example: > > HAWQ200ServiceAdvisor is defined under

Review Request 46977: Hive metastore alerts are present after upgrade/downgrade and enabling security

2016-05-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46977/ --- Review request for Ambari, Jonathan Hurley and Jayush Luniya. Bugs:

Re: Review Request 46971: AMBARI-16248: Create KAFKA service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46971/ --- (Updated May 4, 2016, 2:04 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 46976: AMBARI-16250: Create STORM service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46976/ --- (Updated May 4, 2016, 1:56 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 46963: AMBARI-16243: Create ZOOKEEPER service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46963/ --- (Updated May 4, 2016, 1:59 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 46976: AMBARI-16250: Create STORM service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46976/ --- (Updated May 4, 2016, 1:48 p.m.) Review request for Ambari and Sumit Mohanty.

Review Request 46976: AMBARI-16250: Create STORM service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46976/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-16250

Re: Review Request 46972: Cannot import metrics exported by the export metrics script

2016-05-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46972/#review131662 --- Ship it! Ship It! - Dmitro Lisnichenko On May 4, 2016, 4:06

Review Request 46972: Cannot import metrics exported by the export metrics script

2016-05-04 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46972/ --- Review request for Ambari. Bugs: AMBARI-16249

Review Request 46971: AMBARI-16248: Create KAFKA service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46971/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-16248

Re: Review Request 46971: AMBARI-16248: Create KAFKA service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46971/ --- (Updated May 4, 2016, 12:55 p.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 46963: AMBARI-16243: Create ZOOKEEPER service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46963/ --- (Updated May 4, 2016, 6:56 a.m.) Review request for Ambari and Sumit Mohanty.

Review Request 46963: AMBARI-16243: Create ZOOKEEPER service versions in common-services

2016-05-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46963/ --- Review request for Ambari and Sumit Mohanty. Repository: ambari Description

Review Request 46962: Hive View and Pig View : one user overriding job details of other user in database

2016-05-04 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46962/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar,