Re: Review Request 57350: Kerberos identity reference not working for ranger-audit property in hbase

2017-03-06 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57350/#review168100 --- Ship it! Ship It! - Sebastian Toader On March 7, 2017, 1:13

Re: Review Request 57326: storm DRPC_SERVER kerberos configs duplicate

2017-03-06 Thread wang yaoxin
> On 三月 6, 2017, 4:47 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/0.9.1/configuration/storm-env.xml > > Lines 99-109 (patched) > > > > > > Are these _new_ properties used

Re: Review Request 57326: storm DRPC_SERVER kerberos configs duplicate

2017-03-06 Thread wang yaoxin
> On 三月 6, 2017, 4:47 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/0.9.1/configuration/storm-env.xml > > Lines 99-109 (patched) > > > > > > Are these _new_ properties used

Re: Review Request 57326: storm DRPC_SERVER kerberos configs duplicate

2017-03-06 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57326/ --- (Updated 三月 7, 2017, 2:59 a.m.) Review request for Ambari, Alejandro

Re: Review Request 57346: Value for "User Limit Factor" should be float instead of integer in YARN Queue Manager.

2017-03-06 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57346/ --- (Updated March 7, 2017, 1:02 a.m.) Review request for Ambari, DIPAYAN

Review Request 57350: Kerberos identity reference not working for ranger-audit property in hbase

2017-03-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57350/ --- Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene Chekanskiy,

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-06 Thread Anita Jebaraj
> On March 6, 2017, 9:50 p.m., Di Li wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/RootServiceResponseFactory.java > > Lines 165 (patched) > > > > > > The for loop should not run anyway

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-06 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57168/ --- (Updated March 6, 2017, 10:45 p.m.) Review request for Ambari, Di Li,

Review Request 57346: Value for "User Limit Factor" should be float instead of integer in YARN Queue Manager.

2017-03-06 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57346/ --- Review request for Ambari, DIPAYAN BHOWMICK, Pallav Kulshreshtha, and Sangeeta

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-06 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57168/#review168045 ---

Re: Review Request 57297: Cluster deployment using blueprint with empty configuration doesn't work with stack advisor enabled

2017-03-06 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57297/#review168042 --- Ship it! Ship It! - Di Li On March 3, 2017, 6:59 p.m.,

Re: Review Request 57263: Fix values of ATS config params for HDP stack

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57263/#review168037 ---

Re: Review Request 57287: LogSearch Portal UI Fails on Last Page selection if logs were deleted in the meantime

2017-03-06 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57287/#review168033 --- Ship it! Ship It! - Robert Nettleton On March 3, 2017, 9:23

Re: Review Request 57268: AMBARI-20287 - Filter in Customize Services Page doesn't bring up all properties that matches

2017-03-06 Thread Richard Zang
> On March 4, 2017, 2:33 a.m., Jaimin Jetly wrote: > > ambari-web/app/models/configs/theme/sub_section_tab.js > > Line 84 (original), 84-87 (patched) > > > > > > Whats the reason behind making this computed property

Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-06 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57344/ --- Review request for Ambari, Attila Doroszlai, Andrew Onischuk, Dmytro Sen,

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57324/ --- (Updated March 6, 2017, 12:26 p.m.) Review request for Ambari, Alejandro

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57324/#review168010 --- Ship it! Ship It! - Jonathan Hurley On March 6, 2017, 11:25

Re: Review Request 57281: Script-Based Alert Dispathers support passing more parameters to script

2017-03-06 Thread yao lei
> On 三月 6, 2017, 2:21 p.m., Jonathan Hurley wrote: > > Ship It! > > yao lei wrote: > Hi Jonathan, > Thank you very much. > Would you please help me to commit the patch to trunk if you are free? > > Jonathan Hurley wrote: > Yes, I'm just running a quick test on this patch

Re: Review Request 57089: HDP 3.0 support for Slider with configs, kerberos, widgets, metrics, quicklinks, and themes

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57089/#review168009 --- Fix it, then Ship it!

Re: Review Request 57339: Server startup script keeps waiting even if DB consistency has failed

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57339/#review168003 --- Ship it! Ship It! - Alejandro Fernandez On March 6, 2017,

Re: Review Request 57281: Script-Based Alert Dispathers support passing more parameters to script

2017-03-06 Thread Jonathan Hurley
> On March 6, 2017, 9:21 a.m., Jonathan Hurley wrote: > > Ship It! > > yao lei wrote: > Hi Jonathan, > Thank you very much. > Would you please help me to commit the patch to trunk if you are free? > > Jonathan Hurley wrote: > Yes, I'm just running a quick test on this patch

Re: Review Request 57337: AMBARI-20317 Update stack advisor logic for getting enable atlas hook flag value

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57337/#review168007 --- Ship it! Ship It! - Alejandro Fernandez On March 6, 2017,

Re: Review Request 57295: Atlas service check fails during EU on wire encrypted cluster

2017-03-06 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57295/#review168006 --- Ship it! Ship It! - Dmytro Grinenko On March 6, 2017, 2:57

Re: Review Request 57295: Atlas service check fails during EU on wire encrypted cluster

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57295/#review168005 --- Ship it! Ship It! - Alejandro Fernandez On March 6, 2017,

Re: Review Request 57075: hive-site.xml, hbase-site.xml, etc. are not found in class path for Zeppelin

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57075/#review168002 --- Ship it! Ship It! - Alejandro Fernandez On March 1, 2017,

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57324/#review168001 --- Ship it! Ship It! - Alejandro Fernandez On March 6, 2017,

Re: Review Request 57292: Services shows as Restart required after upgrading from Ambari-2.4.x to 2.5.0.

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57292/#review168000 --- Ship it! Ship It! - Alejandro Fernandez On March 6, 2017,

Re: Review Request 57326: storm DRPC_SERVER kerberos configs duplicate

2017-03-06 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57326/#review167999 ---

Re: Review Request 57292: Services shows as Restart required after upgrading from Ambari-2.4.x to 2.5.0.

2017-03-06 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57292/#review167998 --- Ship it! Ship It! - Sumit Mohanty On March 6, 2017, 3:56

Re: Review Request 57326: storm DRPC_SERVER kerberos configs duplicate

2017-03-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57326/#review167997 ---

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57324/ --- (Updated March 6, 2017, 11:25 a.m.) Review request for Ambari, Alejandro

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Robert Levas
> On March 6, 2017, 9:16 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java > > Lines 220-230 (patched) > > > > > > That's one way of doing it :) > >

Re: Review Request 57281: Script-Based Alert Dispathers support passing more parameters to script

2017-03-06 Thread Jonathan Hurley
> On March 6, 2017, 9:21 a.m., Jonathan Hurley wrote: > > Ship It! > > yao lei wrote: > Hi Jonathan, > Thank you very much. > Would you please help me to commit the patch to trunk if you are free? Yes, I'm just running a quick test on this patch locally and I'll commit it shortly

Re: Review Request 57292: Services shows as Restart required after upgrading from Ambari-2.4.x to 2.5.0.

2017-03-06 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57292/ --- (Updated March 6, 2017, 5:56 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Jonathan Hurley
> On March 6, 2017, 9:16 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java > > Lines 220-230 (patched) > > > > > > That's one way of doing it :) > >

Re: Review Request 57075: hive-site.xml, hbase-site.xml, etc. are not found in class path for Zeppelin

2017-03-06 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57075/#review167988 --- Ship it! Ship It! - Sumit Mohanty On March 1, 2017, 7:35

Re: Review Request 57339: Server startup script keeps waiting even if DB consistency has failed

2017-03-06 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57339/#review167986 --- Ship it! Ship It! - Robert Levas On March 6, 2017, 9:54

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57324/#review167985 --- Ship it! Ship It! - Laszlo Puskas On March 5, 2017, 9:38

Re: Review Request 57339: Server startup script keeps waiting even if DB consistency has failed

2017-03-06 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57339/#review167984 --- Ship it! Ship It! - Laszlo Puskas On March 6, 2017, 2:54

Re: Review Request 57295: Atlas service check fails during EU on wire encrypted cluster

2017-03-06 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57295/ --- (Updated March 6, 2017, 9:57 a.m.) Review request for Ambari, Dmitro

Review Request 57339: Server startup script keeps waiting even if DB consistency has failed

2017-03-06 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57339/ --- Review request for Ambari, Attila Doroszlai, Attila Magyar, Andrew Onischuk,

Re: Review Request 57281: Script-Based Alert Dispathers support passing more parameters to script

2017-03-06 Thread yao lei
> On 三月 6, 2017, 2:21 p.m., Jonathan Hurley wrote: > > Ship It! Hi Jonathan, Thank you very much. Would you please help me to commit the patch to trunk if you are free? - yao --- This is an automatically generated e-mail. To reply,

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Robert Levas
> On March 6, 2017, 9:16 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java > > Lines 220-230 (patched) > > > > > > That's one way of doing it :) > >

Re: Review Request 57281: Script-Based Alert Dispathers support passing more parameters to script

2017-03-06 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57281/#review167979 --- Ship it! Ship It! - Jonathan Hurley On March 3, 2017, 6:56

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57324/#review167976 --- Ship it!

Re: Review Request 57338: ulimit config missing for storm service

2017-03-06 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57338/#review167975 --- Ship it! Ship It! - Dmitro Lisnichenko On March 6, 2017,

Review Request 57337: AMBARI-20317 Update stack advisor logic for getting enable atlas hook flag value

2017-03-06 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57337/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, and Sumit Mohanty.

Re: Review Request 57334: Move breadcrumbs to the separated view

2017-03-06 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57334/#review167972 --- Ship it! Ship It! - Denys Buzhor On March 6, 2017, 1 p.m.,

Re: Review Request 57089: HDP 3.0 support for Slider with configs, kerberos, widgets, metrics, quicklinks, and themes

2017-03-06 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57089/ --- (Updated Март 6, 2017, 11:32 д.п.) Review request for Ambari, Alejandro

Re: Review Request 57324: HBase Master CPU Utilization Alert is in unknown state due to kinit error

2017-03-06 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57324/#review167961 --- Ship it! Ship It! - Sebastian Toader On March 5, 2017,