Re: Review Request 48628: AMBARI-17184: HBase doesn't start because of lacking of variable

2016-07-04 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48628/ --- (Updated July 5, 2016, 4:38 a.m.) Review request for Ambari, Alejandro

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-04 Thread Masahiro Tanaka
> On July 4, 2016, 10:58 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/alerts.json, > > line 43 > > > > > > What about a response? In this case, ZK will response

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-04 Thread Jonathan Hurley
> On July 4, 2016, 6:58 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/alerts.json, > > line 43 > > > > > > What about a response? In this case, ZK will response

Review Request 49618: Hive WebHCat Service Check Fails During Upgrade Due To Missing Configurations Files

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

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-04 Thread Masahiro Tanaka
> On July 4, 2016, 10:58 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/alerts.json, > > line 43 > > > > > > What about a response? In this case, ZK will response

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48972/#review140707 ---

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-04 Thread Masahiro Tanaka
> On June 25, 2016, 10:29 p.m., Nate Cole wrote: > > Latest patch still seems like forced solution. Why not just allow the > > alert definition to provide "data" attribute (or whatever) in json that > > port_alert can use? > > Masahiro Tanaka wrote: > Thanks for reviewing. I've tried to

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-04 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48972/ --- (Updated July 4, 2016, 7:01 p.m.) Review request for Ambari, Florian Barca,

Re: Review Request 49599: Add default value for atlas.authentication.method.ldap.type in common services

2016-07-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49599/#review140689 --- Ship it! Ship It! - Dmitro Lisnichenko On July 4, 2016,

Review Request 49599: Add default value for atlas.authentication.method.ldap.type in common services

2016-07-04 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49599/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Dmytro Sen, and

Review Request 49597: true was removed from stack definition

2016-07-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49597/ --- Review request for Ambari, Jonathan Hurley, Nate Cole, and Sumit Mohanty.

Re: Review Request 49594: 'Restart all' services fails due to dependency on other service components

2016-07-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49594/#review140676 --- Ship it! Ship It! - Dmitro Lisnichenko On July 4, 2016,

Re: Review Request 49596: ambari-server update-host-name failing

2016-07-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49596/#review140674 --- Ship it! Ship It! - Dmitro Lisnichenko On July 4, 2016,

Review Request 49596: ambari-server update-host-name failing

2016-07-04 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49596/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Dmytro Sen.

Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-04 Thread Robert Levas
> On July 1, 2016, 9:45 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py, > > line 303 > > > > > > Why are we hard-coding

Re: Review Request 49594: 'Restart all' services fails due to dependency on other service components

2016-07-04 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49594/#review140664 ---

Re: Review Request 49594: 'Restart all' services fails due to dependency on other service components

2016-07-04 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49594/#review140662 --- Ship it! Ship It! - Vitalyi Brodetskyi On Липень 4, 2016,

Review Request 49594: 'Restart all' services fails due to dependency on other service components

2016-07-04 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49594/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Vitalyi

Re: Review Request 49589: Quicklinks combobox is not present for Oozie

2016-07-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49589/#review140658 --- Ship it! Ship It! - Dmitro Lisnichenko On July 4, 2016,

Review Request 49590: While changing NN, DN directories from UI, proper warning should be present for invalid values

2016-07-04 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49590/ --- Review request for Ambari and Dmytro Sen. Bugs: AMBARI-17550

Re: Review Request 49249: AMBARI-17445: Storm metrics sink: expand metrics name to contain additional informations

2016-07-04 Thread Jungtaek Lim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49249/ --- (Updated 7 4, 2016, 12:23 오후) Review request for Ambari, Aravindan Vijayan,

Re: Review Request 49589: Quicklinks combobox is not present for Oozie

2016-07-04 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49589/#review140649 --- Ship it! Ship It! - Vitalyi Brodetskyi On Липень 4, 2016,

Review Request 49589: Quicklinks combobox is not present for Oozie

2016-07-04 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49589/ --- Review request for Ambari, Andrew Onischuk and Dmitro Lisnichenko. Bugs:

Re: Review Request 49519: Fix Logfeeder visibilities

2016-07-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49519/ --- (Updated July 4, 2016, 10:41 a.m.) Review request for Ambari, Hayat Behlim,

Re: Review Request 49519: Fix Logfeeder visibilities

2016-07-04 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49519/#review140643 --- Ship it! Ship It! - Oliver Szabo On July 1, 2016, 4:46

Re: Review Request 49429: Logsearch shows all the services not only those which are available in the stack

2016-07-04 Thread Miklos Gergely
> On June 30, 2016, 9:55 p.m., Jayush Luniya wrote: > > Can you add more details in the description on how this logic will work? I > > dont have much insights into LogSearch. > > > > Instead of creating HadoopServiceConfig.json.j2 can we not add a > > service_name property in the service

Re: Review Request 49539: AMBARI-17541: LogSearch should also show audit logs from Apache Ranger Solr

2016-07-04 Thread Dharmesh Makwana
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49539/#review140641 --- Ship it! Ship It! - Dharmesh Makwana On July 1, 2016, 11:04

Re: Review Request 49539: AMBARI-17541: LogSearch should also show audit logs from Apache Ranger Solr

2016-07-04 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49539/#review140639 --- Ship it! Ship It! - Oliver Szabo On July 1, 2016, 11:04

Re: Review Request 49539: AMBARI-17541: LogSearch should also show audit logs from Apache Ranger Solr

2016-07-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49539/#review140638 --- Ship it! Ship It! - Miklos Gergely On July 1, 2016, 11:04

Re: Review Request 49535: Ambari Agent memory Leak fix.

2016-07-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49535/#review140631 --- Ship it! Ship It! - Dmitro Lisnichenko On July 4, 2016,

Re: Review Request 49535: Ambari Agent memory Leak fix.

2016-07-04 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49535/ --- (Updated July 4, 2016, 8:45 a.m.) Review request for Ambari and Dmitro

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-07-04 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48487/#review140616 --- Ship it! Ship It! - Sreenath Somarajapuram On June 27,