Re: Review Request 52688: Capacity Scheduler View: Calculating absolute capacity for node labels and showing in sunburst chart

2016-11-01 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52688/#review154514 --- Ship it! Ship It! - Pallav Kulshreshtha On Oct. 19, 2016,

Re: Review Request 52823: For Upload Table on hive view, upload fails when file extension is .json

2016-11-01 Thread Ashwin Rajeev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52823/#review154505 --- Ship it! Ship It! - Ashwin Rajeev On Oct. 13, 2016, 9:06

Re: Review Request 52823: For Upload Table on hive view, upload fails when file extension is .json

2016-11-01 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52823/#review154504 --- Ship it! Ship It! - Pallav Kulshreshtha On Oct. 13, 2016,

Re: Review Request 50561: Hive View Upload table does not support line feed and carriage return in field delimiter

2016-11-01 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50561/#review154502 --- Ship it! Ship It!

Re: Review Request 52822: Ambari Hive View 'Upload Table' does not support UTF8 files with BOM

2016-11-01 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52822/#review154501 --- Ship it! Ship It! - Pallav Kulshreshtha On Oct. 13, 2016,

Re: Review Request 52823: For Upload Table on hive view, upload fails when file extension is .json

2016-11-01 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52823/#review154495 --- Ship it! Ship It! - Gaurav Nagar On Oct. 13, 2016, 9:06

Re: Review Request 52822: Ambari Hive View 'Upload Table' does not support UTF8 files with BOM

2016-11-01 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52822/#review154494 --- Ship it! Ship It! - Gaurav Nagar On Oct. 13, 2016, 8:56

Re: Review Request 50561: Hive View Upload table does not support line feed and carriage return in field delimiter

2016-11-01 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50561/#review154493 --- Ship it! Ship It! - Gaurav Nagar On July 28, 2016, 11:22

Review Request 53364: AMBARI-18767 - E2E Integration for Manage JournalNode Wizard

2016-11-01 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53364/ --- Review request for Ambari, Jaimin Jetly, Zhe (Joe) Wang, and Yusaku Sako.

Re: Review Request 53334: AMBARI-18758. Livy configuration change for upgrade

2016-11-01 Thread Jeff Zhang
> On Nov. 1, 2016, 5:36 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py, > > line 234 > > > > > > Is this meant to account for a different

Re: Review Request 53334: AMBARI-18758. Livy configuration change for upgrade

2016-11-01 Thread Alejandro Fernandez
> On Nov. 1, 2016, 5:36 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py, > > line 234 > > > > > > Is this meant to account for a different

Re: Review Request 53334: AMBARI-18758. Livy configuration change for upgrade

2016-11-01 Thread Jeff Zhang
> On Nov. 1, 2016, 5:36 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py, > > line 234 > > > > > > Is this meant to account for a different

Re: Review Request 53334: AMBARI-18758. Livy configuration change for upgrade

2016-11-01 Thread Jeff Zhang
> On Nov. 1, 2016, 5:36 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py, > > line 234 > > > > > > Is this meant to account for a different

Re: Review Request 51744: SmartSense Mandatory Install

2016-11-01 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51744/#review154483 --- Ship it! Ship It! - Sumit Mohanty On Nov. 1, 2016, 11:55

Re: Review Request 51744: SmartSense Mandatory Install

2016-11-01 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51744/ --- (Updated Nov. 1, 2016, 11:53 p.m.) Review request for Ambari, Jaimin Jetly,

Re: Review Request 53314: Perf: Create PERF stack with mix of HDP core services and dummy services

2016-11-01 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53314/#review154479 ---

Re: Review Request 53314: Perf: Create PERF stack with mix of HDP core services and dummy services

2016-11-01 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53314/#review154476 --- Ship it!

Re: Review Request 53356: Perf: Allow running multiple Ambari Agents on the same host

2016-11-01 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53356/#review154475 --- ambari-agent/conf/windows/service_wrapper.py (line 127)

Review Request 53356: Perf: Allow running multiple Ambari Agents on the same host

2016-11-01 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53356/ --- Review request for Ambari, Aravindan Vijayan, Myroslav Papirkovskyy, Sumit

Re: Review Request 53213: Exporting blueprint from kerberos enabled cluster, exports hardcoded values cluster name and realm in principal_name property

2016-11-01 Thread Amruta Borkar
> On Oct. 31, 2016, 4:27 p.m., Robert Levas wrote: > > This seems dangerous. > > > > I think it would be a better option to make this explicit by using a > > configuration attribute. For example: > > > > ''' > > > > ... > > kerberos_principal > > ... > > > > '''

Re: Review Request 53338: Upgrade Fails From 2.4.2 to 2.5 Due To Existing Role Authorizations

2016-11-01 Thread Robert Levas
On Nov. 1, 2016, 3:32 p.m., Robert Levas wrote: > > Minor question on the method's transactionality... ok.. look like a transaction is in order. - Robert --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 53283: AMBARI-18744: Ambari-server: REST API changes to GET and PUT credential store information

2016-11-01 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53283/#review154442 --- Fix it, then Ship it!

Re: Review Request 53338: Upgrade Fails From 2.4.2 to 2.5 Due To Existing Role Authorizations

2016-11-01 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53338/#review154441 --- Ship it!

Re: Review Request 53283: AMBARI-18744: Ambari-server: REST API changes to GET and PUT credential store information

2016-11-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53283/#review154437 --- Ship it! Ship It! - Robert Levas On Nov. 1, 2016, 1:33

Re: Review Request 53346: AMBARI-18765 - Services with invalid themes hang the UI when you attempt to add them

2016-11-01 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53346/#review154436 --- Ship it! Ship It! - Di Li On Nov. 1, 2016, 6:47 p.m., Tim

Re: Review Request 51815: Introduce conditional dependencies in stack defition to handle blueprint validation gracefully

2016-11-01 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51815/ --- (Updated Nov. 1, 2016, 6:57 p.m.) Review request for Ambari, Shantanu Mundkur,

Review Request 53346: AMBARI-18765 - Services with invalid themes hang the UI when you attempt to add them

2016-11-01 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53346/ --- Review request for Ambari, Di Li and Jayush Luniya. Bugs: AMBARI-18765

Re: Review Request 52369: AMBARI-12263: Support PAM as authentication mechanism for accessing Ambari UI/REST

2016-11-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52369/#review154434 --- Ship it! Ship It! - Robert Levas On Oct. 21, 2016, 10:38

Re: Review Request 53334: AMBARI-18758. Livy configuration change for upgrade

2016-11-01 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53334/#review154422 ---

Re: Review Request 53340: Ambari Database Check failed to Complete Error

2016-11-01 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53340/#review154413 --- Ship it! Ship It! - Jonathan Hurley On Nov. 1, 2016, 11:26

Re: Review Request 53340: Ambari Database Check failed to Complete Error

2016-11-01 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53340/#review154411 --- Ship it! Ship It! - Robert Nettleton On Nov. 1, 2016, 3:26

Review Request 53340: Ambari Database Check failed to Complete Error

2016-11-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53340/ --- Review request for Ambari, Jonathan Hurley, Matt, Nate Cole, and Robert

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-01 Thread Anita Jebaraj
> On Oct. 29, 2016, 5:04 p.m., Attila Doroszlai wrote: > > Nice catch. > > > > Would be nice to use the constants defined in `serverConfiguration` for > > default port and property names. > > > > Even nicer: extract the logic for getting the port (including check for > > SSL) into a function

Re: Review Request 53338: Upgrade Fails From 2.4.2 to 2.5 Due To Existing Role Authorizations

2016-11-01 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53338/#review154399 --- Ship it! Ship It! - Robert Nettleton On Nov. 1, 2016, 1:30

Re: Review Request 53337: ambari-server.pid might not be created

2016-11-01 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53337/#review154398 --- Ship it! Ship It! - Sumit Mohanty On Nov. 1, 2016, 11:13

Re: Review Request 53338: Upgrade Fails From 2.4.2 to 2.5 Due To Existing Role Authorizations

2016-11-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53338/ --- (Updated Nov. 1, 2016, 9:30 a.m.) Review request for Ambari, Jonathan Hurley,

Review Request 53338: Upgrade Fails From 2.4.2 to 2.5 Due To Existing Role Authorizations

2016-11-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53338/ --- Review request for Ambari, Jonathan Hurley, Nate Cole, and Robert Nettleton.

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-01 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53273/#review154392 --- Ship it! Ship It! - Di Li On Nov. 1, 2016, 5:39 a.m., Anita

Re: Review Request 53307: ambari-server upgrade fail (from 2.2.2.0/2.4.0.0/2.4.0.1/2.4.1.0 to 2.4.2.0 )

2016-11-01 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53307/#review154389 --- Ship it! Ship It! - Jonathan Hurley On Oct. 31, 2016, 9:19

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-01 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53273/#review154388 --- Ship it! Ship It! - Tim Thorpe On Nov. 1, 2016, 5:39 a.m.,

Review Request 53337: ambari-server.pid might not be created

2016-11-01 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53337/ --- Review request for Ambari, Sumit Mohanty and Sid Wagle. Bugs: AMBARI-18760

Re: Review Request 53307: ambari-server upgrade fail (from 2.2.2.0/2.4.0.0/2.4.0.1/2.4.1.0 to 2.4.2.0 )

2016-11-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53307/#review154385 --- Ship it! Ship It! - Robert Levas On Oct. 31, 2016, 9:19

Re: Review Request 53307: ambari-server upgrade fail (from 2.2.2.0/2.4.0.0/2.4.0.1/2.4.1.0 to 2.4.2.0 )

2016-11-01 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53307/#review154381 --- Ship it! Ship It! - Dmytro Sen On Ноя. 1, 2016, 1:19 д.п.,