Review Request 51588: AMBARI-18299: Support open files ulimit for ambari-agent

2016-09-01 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51588/ --- Review request for Ambari, Jonathan Hurley and Sumit Mohanty. Bugs:

Review Request 51587: View for Apache HAWQ

2016-09-01 Thread Stuart Pollock
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51587/ --- Review request for Ambari, Alexander Denissov and Lav Jain. Bugs: AMBARI-18297

Review Request 51583: Add ability to add custom grok filters and custom grok patterns with Ambari

2016-09-01 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51583/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty.

Re: Review Request 51570: Enable log search for NIFI_CA component

2016-09-01 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51570/#review147601 --- Ship it! Ship It! - Oliver Szabo On Sept. 1, 2016, 8:44

Review Request 51579: Database Consistency Check Fails With NPE With Missing Service From Stack

2016-09-01 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51579/ --- Review request for Ambari, Jonathan Hurley and Sumit Mohanty. Bugs:

Review Request 51578: AMBARI-18071: Ambari Files View needs to have ability to load security configurations

2016-09-01 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51578/ --- Review request for Ambari, Di Li and DIPAYAN BHOWMICK. Bugs: AMBARI-18071

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-09-01 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/ --- (Updated Sept. 1, 2016, 6:11 p.m.) Review request for Ambari, Dmytro Sen,

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-09-01 Thread Aravindan Vijayan
> On Sept. 1, 2016, 4:59 a.m., Alejandro Fernandez wrote: > > ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/host_info.py, > > line 266 > > > > > > Where is allowed_disks populated? In line 274

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51557/#review147567 --- Ship it! Ship It! - Sumit Mohanty On Sept. 1, 2016, 2:59

Re: Review Request 51370: DB consistency check should tolerate services with no configuration

2016-09-01 Thread Balazs Bence Sari
+ Jayush Hey Nate, It has not yet been committed. @Jayush: Should we commit it to trunk only or to 2.5 as well? Bence From: Nate Cole > on behalf of Nate Cole > Reply-To: Nate

Re: Review Request 51238: Implementation for AMBARI-15538: Support service-specific repo for add-on services

2016-09-01 Thread Balázs Bence Sári
> On Sept. 1, 2016, 2:10 p.m., Tim Thorpe wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/RepoUtil.java, > > line 138 > > > > > > The Optional class is only in JDK8. Currently Ambari is

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-09-01 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51452/#review147561 --- Ship it! Ship It! - Sreenath Somarajapuram On Sept. 1,

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Vitalyi Brodetskyi
> On Вер. 1, 2016, 2:13 після полудня, Sumit Mohanty wrote: > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java, > > line 269 > > > > > > Can we add a unit test? Done - Vitalyi

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51557/ --- (Updated Вер. 1, 2016, 2:59 після полудня) Review request for Ambari, Andrew

Re: Review Request 51238: Implementation for AMBARI-15538: Support service-specific repo for add-on services

2016-09-01 Thread Tim Thorpe
> On Aug. 29, 2016, 5:27 p.m., Nate Cole wrote: > > I was under the impression that applying an m-pack would DIRECTLY update > > repoinfo.xml on the filesystem. Has the design changed since then? > > > > What is the purpose of carrying service version around with the repo? When > > updating

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51557/#review147558 ---

Re: Review Request 51238: Implementation for AMBARI-15538: Support service-specific repo for add-on services

2016-09-01 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51238/#review147557 ---

Re: Review Request 51575: Provide stack-advisor support for Microsoft-R service

2016-09-01 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51575/#review147556 ---

Re: Review Request 51562: AMBARI-18289 : Invalid negative values for some AMS metrics

2016-09-01 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51562/#review147555 ---

Review Request 51575: Provide stack-advisor support for Microsoft-R service

2016-09-01 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51575/ --- Review request for Ambari, Balázs Bence Sári, Jayush Luniya, Sumit Mohanty, and

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-09-01 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51452/ --- (Updated Sept. 1, 2016, 1:28 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-09-01 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51452/ --- (Updated Sept. 1, 2016, 11:36 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 51572: Ambari Server Start/Stop fails on Centos 7.1+

2016-09-01 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51572/#review147550 --- Ship it! Ship It! - Dmitro Lisnichenko On Sept. 1, 2016,

Review Request 51572: Ambari Server Start/Stop fails on Centos 7.1+

2016-09-01 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51572/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-18294

Review Request 51570: Enable log search for NIFI_CA component

2016-09-01 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51570/ --- Review request for Ambari, Jayush Luniya, Oliver Szabo, Robert Nettleton, and