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

2016-05-03 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-03 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-03 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,

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-05-03 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/ --- (Updated May 4, 2016, 6:20 a.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-05-03 Thread Jayush Luniya
> On April 29, 2016, 9:53 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.6.x.2.4/metainfo.xml, > > line 1 > > > > > > We shouldnt name this as 1.6.x.2.4. Just SPARK/1.6.0. I do

Re: Review Request 46833: AMBARI-16182 Add new component level identity for RANGER_KMS in kerberos.json for stack 2.5

2016-05-03 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46833/#review131626 --- Ship it! Ship It! - Jayush Luniya On May 2, 2016, 12:55 p.m.

Re: Review Request 42155: AMBARI-14605 '[RAM_DISK]' configured in dfs.datanode.data.dir fails to validate

2016-05-03 Thread yao lei
> On 五月 4, 2016, 4:14 a.m., Matt wrote: > > You may close this review as it has been submitted already. ok,thanks,Matt - yao --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42155/#review13162

Re: Review Request 42155: AMBARI-14605 '[RAM_DISK]' configured in dfs.datanode.data.dir fails to validate

2016-05-03 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42155/#review131621 --- You may close this review as it has been submitted already. - Mat

Re: Review Request 46959: AMBARI-16240: Ambari operation history: Syntax error in Ambari-DDL-Postgres-CREATE.sql causes ambari-server setup to fail in 2.4.0.0-489

2016-05-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46959/#review131619 --- Ship it! Ship It! - Sumit Mohanty On May 4, 2016, 3:57 a.m.,

Re: Review Request 46959: AMBARI-16240: Ambari operation history: Syntax error in Ambari-DDL-Postgres-CREATE.sql causes ambari-server setup to fail in 2.4.0.0-489

2016-05-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46959/ --- (Updated May 3, 2016, 8:57 p.m.) Review request for Ambari, Ajit Kumar and Sumi

Re: Review Request 46959: AMBARI-16240: Ambari operation history: Syntax error in Ambari-DDL-Postgres-CREATE.sql causes ambari-server setup to fail in 2.4.0.0-489

2016-05-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46959/#review131618 --- Ship it! Ship It! - Sumit Mohanty On May 4, 2016, 3:43 a.m.,

Review Request 46959: AMBARI-16240: Ambari operation history: Syntax error in Ambari-DDL-Postgres-CREATE.sql causes ambari-server setup to fail in 2.4.0.0-489

2016-05-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46959/ --- Review request for Ambari, Ajit Kumar and Sumit Mohanty. Bugs: AMBARI-16240

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

2016-05-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46947/#review131611 --- ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/

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

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

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

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

Re: Review Request 45867: Add shiro.ini in Ambari Zeppelin service config

2016-05-03 Thread Jayush Luniya
> On April 7, 2016, 5:23 p.m., Jayush Luniya wrote: > > Ship It! Committed. Please close review commit 2c5de15b3dd18453f67a11194d9ef7d3dab46e45 Author: Jayush Luniya Date: Tue May 3 17:53:56 2016 -0700 AMBARI-15758: Add shiro.ini in Ambari Zeppelin service config (Renjith Kamath via jl

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

2016-05-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46947/#review131603 --- ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/con

Re: Review Request 46953: AMBARI-16239: Incorrect warning for not selecting nfs_gateway_hosts

2016-05-03 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46953/#review131597 --- Ship it! Ship It! - Zhe (Joe) Wang On May 4, 2016, 12:36 a.m

Review Request 46953: AMBARI-16239: Incorrect warning for not selecting nfs_gateway_hosts

2016-05-03 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46953/ --- Review request for Ambari, Andriy Babiichuk and Zhe (Joe) Wang. Bugs: AMBARI-16

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

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

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

2016-05-03 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46944/#review131590 --- Ship it! Ship It! - Ajit Kumar On May 3, 2016, 7:36 p.m., Ol

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-03 Thread bhuvnesh chaudhary
> On May 3, 2016, 11:14 p.m., Matt wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/namenode_ha_utils.py, > > line 142 > > > > > > Is there a possibility where active_namenodes can be

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-03 Thread Matt
> On May 3, 2016, 4:14 p.m., Matt wrote: > > Discussed this in person: This function is not used anywhere else, besides PXF service. Logging would be taken care of by PXF service component. - Matt --- This is an automatically generated

Review Request 46950: AMBARI-16231: mapreduce.jobhistory.http.policy & mapreduce.jobhistory.webapp.https.address property are not respected

2016-05-03 Thread Vishal Ghugare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46950/ --- Review request for Ambari and Andrew-Duplicate-Accct-Inactiv Konwinski. Bugs: A

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-03 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46949/#review131580 --- Ship it! ambari-common/src/main/python/resource_management/lib

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

2016-05-03 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/#review131579 --- The functionality of ServiceAdvisor works as expected. However mor

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-03 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46949/#review131578 --- Ship it! Ship It! - Goutam Tadi On May 3, 2016, 11:05 p.m.,

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

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

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

2016-05-03 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46927/#review131570 --- Ship it! Ship It! - Jaimin Jetly On May 3, 2016, 4:52 a.m.,

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

2016-05-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46947/ --- Review request for Ambari, Sumit Mohanty and Swapan Shridhar. Bugs: AMBARI-1623

Re: Review Request 46545: AMBARI-16029: Ambari version history - Create DB table, constraints and sequence id.

2016-05-03 Thread Nahappan Somasundaram
> On May 3, 2016, 2:33 p.m., Jonathan Hurley wrote: > > The upgrade catalog stuff will be in a separate Jira? Yes. - Nahappan --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46545/#review1315

Re: Review Request 46545: AMBARI-16029: Ambari version history - Create DB table, constraints and sequence id.

2016-05-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46545/#review131562 --- Ship it! The upgrade catalog stuff will be in a separate Jira?

Re: Review Request 46931: Updated grok lib version(0.1.1 to 0.1.4) to get latest fixes.

2016-05-03 Thread Robert Nettleton
> On May 3, 2016, 8:28 p.m., Sumit Mohanty wrote: > > Ship It! I've just pushed this patch to trunk. Could the submitter please close out this ReviewBoard as "Submitted"? Thanks. - Robert --- This is an automatically generated e-mai

Re: Review Request 46931: Updated grok lib version(0.1.1 to 0.1.4) to get latest fixes.

2016-05-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46931/#review131546 --- Ship it! Ship It! - Sumit Mohanty On May 3, 2016, 11:42 a.m.

Re: Review Request 46931: Updated grok lib version(0.1.1 to 0.1.4) to get latest fixes.

2016-05-03 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46931/#review131545 --- Ship it! Ship It! - Don Bosco Durai On May 3, 2016, 11:42 a.

Re: Review Request 46545: AMBARI-16029: Ambari version history - Create DB table, constraints and sequence id.

2016-05-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46545/ --- (Updated May 3, 2016, 1:09 p.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 46545: AMBARI-16029: Ambari version history - Create DB table, constraints and sequence id.

2016-05-03 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46545/#review131542 --- Ship it! Ship It! - Ajit Kumar On May 3, 2016, 4:33 p.m., Na

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

2016-05-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46944/ --- Review request for Ambari, Jonathan Hurley, Laszlo Puskas, and Sebastian Toader.

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

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

Re: Review Request 46853: clean up import * : Accumulo, Ambari-metrics, Atlas, Falcon, Flume and Ganglia

2016-05-03 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46853/ --- (Updated May 3, 2016, 6:12 p.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 46853: clean up import * : Accumulo, Ambari-metrics, Atlas, Falcon, Flume and Ganglia

2016-05-03 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46853/ --- (Updated May 3, 2016, 6:11 p.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 46738: Converting cluster to Kerberos the views should automatically adjust and work prior to the kerberization.

2016-05-03 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46738/#review131535 --- contrib/views/utils/src/main/java/org/apache/ambari/view/utils/am

Re: Review Request 46860: Move RM wizard should update HAWQ related parameters

2016-05-03 Thread jun aoki
> On April 30, 2016, 12:21 a.m., bhuvnesh chaudhary wrote: > > ambari-web/app/controllers/main/service/reassign/step4_controller.js, line > > 85 > > > > > > Unless HAWQ is using YARN as the scheduler, HAWQ may still

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

2016-05-03 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46941/#review131531 --- Ship it! Ship It! - Oliver Szabo On May 3, 2016, 4:42 p.m.,

Re: Review Request 46853: clean up import * : Accumulo, Ambari-metrics, Atlas, Falcon, Flume and Ganglia

2016-05-03 Thread Juanjo Marron
> On May 3, 2016, 12:32 a.m., Matt wrote: > > Ship It! Waiting for Hadoop QA to have new patch available with installation and service checks tests performed Will upload it soon. Thanks for the review - Juanjo --- This is an automatica

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

2016-05-03 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46941/ --- Review request for Ambari, Don Bosco Durai, Miklos Gergely, Oliver Szabo, and Su

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

2016-05-03 Thread Tim Thorpe
> On May 2, 2016, 11:07 p.m., Lav Jain wrote: > > ambari-server/src/main/resources/stacks/stack_advisor.py, line 644 > > > > > > This method is not being called because the corresponding method in > > 2.0.6 stack ad

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

2016-05-03 Thread Lav Jain
> On May 2, 2016, 11:07 p.m., Lav Jain wrote: > > ambari-server/src/main/resources/stacks/stack_advisor.py, line 644 > > > > > > This method is not being called because the corresponding method in > > 2.0.6 stack ad

Re: Review Request 46545: AMBARI-16029: Ambari version history - Create DB table, constraints and sequence id.

2016-05-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46545/ --- (Updated May 3, 2016, 9:33 a.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 46925: Write out new log property files under /etc/hive2/conf

2016-05-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46925/#review131513 --- ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/

Re: Review Request 46545: AMBARI-16029: Ambari version history - Create DB table, constraints and sequence id.

2016-05-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46545/#review131508 --- ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql (lin

Re: Review Request 46925: Write out new log property files under /etc/hive2/conf

2016-05-03 Thread Sumit Mohanty
> On May 3, 2016, 7:32 a.m., Swapan Shridhar wrote: > > ambari-server/src/test/python/stacks/2.5/configs/hsi_default.json, line 52 > > > > > > is \n intentional ? Yes to simulate new lines in content properties in co

Re: Review Request 46925: Write out new log property files under /etc/hive2/conf

2016-05-03 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46925/ --- (Updated May 3, 2016, 2:29 p.m.) Review request for Ambari, Nahappan Somasundar

Re: Review Request 46925: Write out new log property files under /etc/hive2/conf

2016-05-03 Thread Sumit Mohanty
> On May 3, 2016, 2 p.m., Nahappan Somasundaram wrote: > > ambari-server/src/test/python/stacks/2.5/HIVE/test_hive_server_int.py, line > > 270 > > > > > > Is the newline intentional? Yes to simulate new lines in co

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

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

Re: Review Request 46925: Write out new log property files under /etc/hive2/conf

2016-05-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46925/#review131499 --- ambari-server/src/test/python/stacks/2.5/HIVE/test_hive_server_in

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

2016-05-03 Thread Tim Thorpe
> On May 2, 2016, 11:07 p.m., Lav Jain wrote: > > ambari-server/src/main/resources/stacks/stack_advisor.py, line 644 > > > > > > This method is not being called because the corresponding method in > > 2.0.6 stack ad

Re: Review Request 46899: Disable alternate user search functionality by default

2016-05-03 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46899/#review131493 --- Ship it! Ship It! - Laszlo Puskas On May 2, 2016, 8:01 p.m.,

Re: Review Request 46899: Disable alternate user search functionality by default

2016-05-03 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46899/#review131492 --- ambari-server/src/main/java/org/apache/ambari/server/configuratio

Re: Review Request 46899: Disable alternate user search functionality by default

2016-05-03 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46899/#review131487 --- Ship it! Ship It! - Sandor Magyari On May 2, 2016, 8:01 p.m.

Re: Review Request 46899: Disable alternate user search functionality by default

2016-05-03 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46899/#review131486 --- Ship it! Ship It! - Balázs Bence Sári On May 2, 2016, 8:01 p

Review Request 46931: Updated grok lib version(0.1.1 to 0.1.4) to get latest fixes.

2016-05-03 Thread Hayat Behlim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46931/ --- Review request for Ambari. Bugs: AMBARI-16211 https://issues.apache.org/jir

Re: Review Request 46899: Disable alternate user search functionality by default

2016-05-03 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46899/#review131475 --- Ship it! Ship It! - Robert Levas On May 2, 2016, 4:01 p.m.,

Re: Review Request 46899: Disable alternate user search functionality by default

2016-05-03 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46899/#review131471 --- Ship it! Ship It! - Daniel Gergely On máj. 2, 2016, 8:01 du,

Re: Review Request 46925: Write out new log property files under /etc/hive2/conf

2016-05-03 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46925/#review131466 --- ambari-server/src/test/python/stacks/2.5/configs/hsi_default.json

Re: Review Request 46925: Write out new log property files under /etc/hive2/conf

2016-05-03 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46925/#review131465 --- Ship it! Ship It! - Swapan Shridhar On May 3, 2016, 4:27 a.m