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

2016-10-13 Thread Vishal Ghugare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52369/ --- (Updated Oct. 13, 2016, 8:25 p.m.) Review request for Ambari, Alejandro

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

2016-10-13 Thread Vishal Ghugare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52369/ --- (Updated Oct. 13, 2016, 7:58 p.m.) Review request for Ambari, Alejandro

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

2016-10-13 Thread Vishal Ghugare
> On Oct. 7, 2016, 10:55 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/Users.java, > > line 817 > > > > > > Since this is called each time Ambari starts

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

2016-10-13 Thread Vishal Ghugare
> On Oct. 3, 2016, 1:17 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/Users.java, > > lines 955-960 > > > > > > roles may be set today, but in the future

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-13 Thread Nate Cole
> On Oct. 13, 2016, 7:10 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > line 512 > > > > > > I'm curious - why use the configuration property

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

2016-10-13 Thread Vishal Ghugare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52369/ --- (Updated Oct. 13, 2016, 6:45 p.m.) Review request for Ambari, Alejandro

Review Request 52851: rpm command broken on utility project

2016-10-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52851/ --- Review request for Ambari, Dmytro Grinenko and Sumit Mohanty. Repository:

Review Request 52850: Icons don't display in Windows 10 Microsoft Edge or Internet Explorer 11

2016-10-13 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52850/ --- Review request for Ambari, Jaimin Jetly, Zhe (Joe) Wang, Richard Zang, Xi Wang,

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-13 Thread Amarnath reddy pappu
> On Oct. 13, 2016, 6:20 p.m., Alejandro Fernandez wrote: > > Ship It! thank you - Amarnath reddy --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review152549

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-13 Thread Amarnath reddy pappu
> On Oct. 13, 2016, 11:10 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > line 512 > > > > > > I'm curious - why use the configuration property

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review152595 ---

Re: Review Request 52837: Hive Rolling Upgrade Is No Longer Supported In Ambari

2016-10-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52837/#review152594 --- Ship it! Ship It! - Nate Cole On Oct. 13, 2016, 1:17 p.m.,

Re: Review Request 52843: BP format does not generate property oozie.service.JPAService.jdbc.url

2016-10-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52843/#review152579 --- Ship it! Ship It! - Alejandro Fernandez On Oct. 13, 2016,

Review Request 52843: BP format does not generate property oozie.service.JPAService.jdbc.url

2016-10-13 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52843/ --- Review request for Ambari, Alejandro Fernandez, Sumit Mohanty, and Sid Wagle.

Review Request 52842: AMBARI-18593 : Provide ability to use downsampling function on certain metrics like client side topN

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

Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-13 Thread Amarnath reddy pappu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Nate Cole, and

Re: Review Request 52837: Hive Rolling Upgrade Is No Longer Supported In Ambari

2016-10-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52837/#review152540 --- Ship it! Ship It! - Alejandro Fernandez On Oct. 13, 2016,

Re: Review Request 52837: Hive Rolling Upgrade Is No Longer Supported In Ambari

2016-10-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52837/#review152536 ---

Review Request 52837: Hive Rolling Upgrade Is No Longer Supported In Ambari

2016-10-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52837/ --- Review request for Ambari, Alejandro Fernandez, Nate Cole, and Robert Levas.

Re: Review Request 52833: RegionServer Registration Checks Fail During Upgrade If rDNS is Not Enabled

2016-10-13 Thread Alejandro Fernandez
> On Oct. 13, 2016, 5:02 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/upgrade.py, > > line 64 > > > > > > Let's decrease the sleep time to 10

Re: Review Request 52833: RegionServer Registration Checks Fail During Upgrade If rDNS is Not Enabled

2016-10-13 Thread Nate Cole
> On Oct. 13, 2016, 1:02 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/upgrade.py, > > line 64 > > > > > > Let's decrease the sleep time to 10

Re: Review Request 52833: RegionServer Registration Checks Fail During Upgrade If rDNS is Not Enabled

2016-10-13 Thread Jonathan Hurley
> On Oct. 13, 2016, 1:02 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/upgrade.py, > > line 64 > > > > > > Let's decrease the sleep time to 10

Re: Review Request 52833: RegionServer Registration Checks Fail During Upgrade If rDNS is Not Enabled

2016-10-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52833/#review152531 --- Fix it, then Ship it!

Re: Review Request 52833: RegionServer Registration Checks Fail During Upgrade If rDNS is Not Enabled

2016-10-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52833/#review152529 --- Ship it! Ship It! - Nate Cole On Oct. 13, 2016, 11:19 a.m.,

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52826/#review152527 --- Ship it! Ship It! - Dmitro Lisnichenko On Oct. 13, 2016,

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52826/ --- (Updated Oct. 13, 2016, 4:41 p.m.) Review request for Ambari and Vitalyi

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Jonathan Hurley
> On Oct. 13, 2016, 9:40 a.m., Sumit Mohanty wrote: > > ambari-project/pom.xml, line 516 > > > > > > Why 1.5C? > > Jonathan Hurley wrote: > 1.5C means 1.5 x # of cores - so that lowered powered machines (like

Re: Review Request 52747: Migration to Bootstrap3 framework

2016-10-13 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52747/#review152522 --- Ship it! Ship It! - Yusaku Sako On Oct. 13, 2016, 1:48

Re: Review Request 52691: Provision actions to happen based only on specified dependencies

2016-10-13 Thread Sandor Magyari
> On Oct. 11, 2016, 6:37 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java, > > line 410 > > > > > > Does this only affect fresh install

Re: Review Request 52691: Provision actions to happen based only on specified dependencies

2016-10-13 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52691/ --- (Updated Oct. 13, 2016, 3:50 p.m.) Review request for Ambari, Alejandro

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Andrew Onischuk
> On Oct. 13, 2016, 1:40 p.m., Sumit Mohanty wrote: > > ambari-project/pom.xml, line 515 > > > > > > Should this be true? this part will be committed separately. This small change significantly decreases time on

Review Request 52833: RegionServer Registration Checks Fail During Upgrade If rDNS is Not Enabled

2016-10-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52833/ --- Review request for Ambari, Alejandro Fernandez and Nate Cole. Bugs:

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Andrew Onischuk
> On Oct. 13, 2016, 1:40 p.m., Sumit Mohanty wrote: > > ambari-project/pom.xml, line 516 > > > > > > Why 1.5C? > > Jonathan Hurley wrote: > 1.5C means 1.5 x # of cores - so that lowered powered machines (like

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2016-10-13 Thread Zsombor Gegesy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52785/ --- (Updated Oct. 13, 2016, 2:52 p.m.) Review request for Ambari and Sandor

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Jonathan Hurley
> On Oct. 13, 2016, 9:40 a.m., Sumit Mohanty wrote: > > ambari-project/pom.xml, line 516 > > > > > > Why 1.5C? 1.5C means 1.5 x # of cores - so that lowered powered machines (like build machines) don't spawn too

Re: Review Request 52829: HCat client install during Ambari install wizard

2016-10-13 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52829/#review152502 --- Ship it! Ship It! - Dmytro Sen On Окт. 13, 2016, 2:43 п.п.,

Review Request 52829: HCat client install during Ambari install wizard

2016-10-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52829/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-18589

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2016-10-13 Thread Zsombor Gegesy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52785/ --- (Updated Oct. 13, 2016, 2:24 p.m.) Review request for Ambari and Sandor

Re: Review Request 52825: AMBARI-18585 Set _HOST for Hbase Master principal in Kerberos enabled Ranger Hbase Plugin

2016-10-13 Thread Velmurugan Periasamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52825/#review152500 --- Ship it! Ship It! - Velmurugan Periasamy On Oct. 13, 2016,

Re: Review Request 52747: Migration to Bootstrap3 framework

2016-10-13 Thread Aleksandr Kovalenko
> On Окт. 12, 2016, 8:35 п.п., Yusaku Sako wrote: > > Thanks for moving the ambari bootstrap override as unminified, non-vendor > > style. > > Can we move the Roboto font files back to vendor, though? done - Aleksandr --- This is an

Re: Review Request 52747: Migration to Bootstrap3 framework

2016-10-13 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52747/ --- (Updated Окт. 13, 2016, 1:48 п.п.) Review request for Ambari, Alexandr

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52826/#review152494 --- ambari-project/pom.xml (line 515)

Re: Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52826/#review152492 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 13, 2016,

Review Request 52826: Increase forkCount for Ambari Server Unit Tests

2016-10-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52826/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-18586

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

2016-10-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51815/#review152489 ---

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

2016-10-13 Thread Nate Cole
> On Oct. 7, 2016, 3:39 p.m., Nate Cole wrote: > > This code is breaking tests, and has been reverted from trunk. Please fix > > the tests, and also fix the formatting. Ambari is standardized on two > > space tabs. DependencyConditionInfo is a undocumented and unformatted > > class. In

Review Request 52825: AMBARI-18585 Set _HOST for Hbase Master principal in Kerberos enabled Ranger Hbase Plugin

2016-10-13 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52825/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya,

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2016-10-13 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52785/#review152483 --- Ship it! ambari-server/src/main/python/setupAgent.py (line

Re: Review Request 52791: Getting errors with max length 1000byte, when using Mysql db with charset UTF8

2016-10-13 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52791/#review152481 --- Ship it! Ship It! - Myroslav Papirkovskyy On Жов. 12, 2016,

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

2016-10-13 Thread Ashwin Rajeev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50561/#review152475 --- Ship it! Ship It! - Ashwin Rajeev On July 28, 2016, 11:22

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

2016-10-13 Thread Ashwin Rajeev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52822/#review152474 --- Ship it! Ship It! - Ashwin Rajeev On Oct. 13, 2016, 8:56

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2016-10-13 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52785/#review152470 --- Ship it! Ship It! - Laszlo Puskas On Oct. 12, 2016, 2:36

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

2016-10-13 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52823/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav Kulshreshtha,

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

2016-10-13 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52822/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav Kulshreshtha,

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2016-10-13 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52785/#review152466 --- Ship it! Could you please run 'mvn clean test

Re: Review Request 52792: AMBARI-18581 Oozie start script ignores sysprep_skip_copy_oozie_share_lib_to_hdfs flag

2016-10-13 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52792/#review152462 --- Ship it! Ship It! - Sebastian Toader On Oct. 12, 2016, 7:51