Re: Review Request 53142: Webhcat server start failed during EU with BindException

2016-10-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53142/#review153767 --- Ship it! Ship It! - Robert Levas On Oct. 24, 2016, 1:16 p.m.

Review Request 53158: AMBARI-18687: Ambari-agent - CLI utility to support storing passwords using hadoop credential store

2016-10-24 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53158/ --- Review request for Ambari, Jonathan Hurley, Mahadev Konar, Nate Cole, Robert Lev

Re: Review Request 53106: Move HAWQ and PXF RCO from stacks to common-services

2016-10-24 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53106/#review153764 --- Ship it! - Lav Jain On Oct. 21, 2016, 10:46 p.m., Matt wrote:

Re: Review Request 53087: Log Search schema meta data loader fix

2016-10-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53087/ --- (Updated Oct. 24, 2016, 10:14 p.m.) Review request for Ambari, Oliver Szabo, Ro

Re: Review Request 53144: Workflow Designer View: Tooltip for Edit button shows "Back"

2016-10-24 Thread Sangeeta Ravindran
> On Oct. 24, 2016, 9:34 p.m., Di Li wrote: > > contrib/views/wfmanager/src/main/resources/ui/app/templates/components/job-details.hbs, > > line 55 > > > > > > can this be externalized to a i18n file instead of hardc

Re: Review Request 51724: Add Kerberos HTTP SPNEGO authentication support to Ambari Metrics Monitor

2016-10-24 Thread Qin Liu
> On Sept. 22, 2016, 7:54 p.m., Robert Levas wrote: > > ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/spnego_kerberos_auth.py, > > line 33 > > > > > > See `curl_krb_request` > > Dmytro Sen wrote

Re: Review Request 53143: Workflow Designer View: Folder hint does not get overwritten when you type folder name when creating folder in File Browser

2016-10-24 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53143/#review153743 --- Ship it! Ship It! - Di Li On Oct. 24, 2016, 6:32 p.m., Sange

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
> On Oct. 24, 2016, 6:57 p.m., Alexandr Antonenko wrote: > > Ship It! > > Anita Jebaraj wrote: > Thank you...Can you please help in pushing the code? > > Alexandr Antonenko wrote: > done Thank you - Anita --- This is an automa

Re: Review Request 53144: Workflow Designer View: Tooltip for Edit button shows "Back"

2016-10-24 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53144/#review153740 --- contrib/views/wfmanager/src/main/resources/ui/app/templates/compo

Re: Review Request 53126: Workflow Designer View: When you execute a workflow designer view instance, it does not load immediately unless you return to the dashboard.

2016-10-24 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53126/#review153739 --- Ship it! Ship It! - Di Li On Oct. 24, 2016, 1:33 a.m., Anita

Re: Review Request 53142: Webhcat server start failed during EU with BindException

2016-10-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53142/#review153738 --- Ship it! Ship It! - Nate Cole On Oct. 24, 2016, 1:16 p.m., J

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Alexandr Antonenko
> On Oct. 24, 2016, 6:57 p.m., Alexandr Antonenko wrote: > > Ship It! > > Anita Jebaraj wrote: > Thank you...Can you please help in pushing the code? done - Alexandr --- This is an automatically generated e-mail. To reply, visit: h

Re: Review Request 53109: Manage Journal Node Wizard Initial UI

2016-10-24 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53109/#review153731 --- Ship it! Ship It! - Zhe (Joe) Wang On Oct. 22, 2016, 12:09 a

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53140/#review153724 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 24, 2016, 4:5

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
> On Oct. 24, 2016, 6:57 p.m., Alexandr Antonenko wrote: > > Ship It! Thank you...Can you please help in pushing the code? - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53140/#revie

Re: Review Request 53075: Fix JSHint errors in Workflow Manager view

2016-10-24 Thread Sangeeta Ravindran
> On Oct. 21, 2016, 7:59 p.m., Venkat Ranganathan wrote: > > There are more changes we are working on (introducing Coordinator, bundle > > etc) and some further refactoring (the use of validator etc). Let me > > review thi a bit more carefully if there are intersections Thank you Venkat. Do

Re: Review Request 53143: Workflow Designer View: Folder hint does not get overwritten when you type folder name when creating folder in File Browser

2016-10-24 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53143/ --- (Updated Oct. 24, 2016, 6:32 p.m.) Review request for Ambari, Di Li, Venkat Ran

Review Request 53143: Workflow Designer View: Folder hint does not get overwritten when you type folder name when creating folder in File Browser

2016-10-24 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53143/ --- Review request for Ambari, Di Li, Venkat Ranganathan, and Yusaku Sako. Bugs: AM

Re: Review Request 53066: Preview - HDP-2.5 installation allows ZKFC to advertise version

2016-10-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53066/#review153710 --- ambari-server/src/main/java/org/apache/ambari/server/events/Stack

Review Request 53144: Workflow Designer View: Tooltip for Edit button shows "Back"

2016-10-24 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53144/ --- Review request for Ambari, Di Li, Venkat Ranganathan, and Yusaku Sako. Bugs: AM

Review Request 53142: Webhcat server start failed during EU with BindException

2016-10-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53142/ --- Review request for Ambari, Dmitro Lisnichenko, Nate Cole, and Robert Levas. Bug

Re: Review Request 50803: [Ambari-17981] Integrate Druid With Ambari

2016-10-24 Thread Slim Bouguerra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50803/#review153707 --- Ship it! Ship It! - Slim Bouguerra On Oct. 22, 2016, 1:21 a.

Re: Review Request 53101: While syncing with LDAP, username collisions should be handled based on configuration value

2016-10-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53101/#review153703 --- Ship it! Just some minor comments around the configurations a

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53140/ --- (Updated Oct. 24, 2016, 4:58 p.m.) Review request for Ambari, Alexandr Antonenk

Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53140/ --- Review request for Ambari, Alexandr Antonenko and Di Li. Bugs: AMBARI-18638

Re: Review Request 53066: Preview - HDP-2.5 installation allows ZKFC to advertise version

2016-10-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53066/ --- (Updated Oct. 24, 2016, 7:35 p.m.) Review request for Ambari, Jonathan Hurley a

Re: Review Request 53101: While syncing with LDAP, username collisions should be handled based on configuration value

2016-10-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53101/ --- (Updated Oct. 24, 2016, 12:34 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 53066: Preview - HDP-2.5 installation allows ZKFC to advertise version

2016-10-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53066/ --- (Updated Oct. 24, 2016, 7:34 p.m.) Review request for Ambari, Jonathan Hurley a

Re: Review Request 53066: Preview - HDP-2.5 installation allows ZKFC to advertise version

2016-10-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53066/ --- (Updated Oct. 24, 2016, 7:30 p.m.) Review request for Ambari, Jonathan Hurley a

Review Request 53139: Zeppelin install not correctly setting user and group for notebook directory

2016-10-24 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53139/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Jayush Luniya,

Re: Review Request 52420: Ambari Status commands should enforce a timeout < heartbeat interval

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

Re: Review Request 52420: Ambari Status commands should enforce a timeout < heartbeat interval

2016-10-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52420/#review153693 --- ambari-agent/src/main/python/ambari_agent/ActionQueue.py (line 79

Re: Review Request 52420: Ambari Status commands should enforce a timeout < heartbeat interval

2016-10-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52420/ --- (Updated Oct. 24, 2016, 3:18 p.m.) Review request for Ambari, Sumit Mohanty, Si

Re: Review Request 52420: Ambari Status commands should enforce a timeout < heartbeat interval

2016-10-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52420/ --- (Updated Oct. 24, 2016, 3:16 p.m.) Review request for Ambari, Sumit Mohanty, Si

Re: Review Request 53135: Fix constraint violations in adminprincipal table when installing database schema

2016-10-24 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53135/#review153691 --- Ship it! Ship It! - Laszlo Puskas On Oct. 24, 2016, 2:30 p.m

Re: Review Request 53135: Fix constraint violations in adminprincipal table when installing database schema

2016-10-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53135/#review153690 --- Ship it! Ship It! - Oliver Szabo On Oct. 24, 2016, 2:30 p.m.

Re: Review Request 52930: Ambari Metrics doesn't use SPNEGO to authenticate

2016-10-24 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52930/ --- (Updated Окт. 24, 2016, 2:31 п.п.) Review request for Ambari, Aravindan Vijayan

Re: Review Request 53135: Fix constraint violations in adminprincipal table when installing database schema

2016-10-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53135/#review153688 --- Ship it! Ship It! - Nate Cole On Oct. 24, 2016, 10:30 a.m.,

Re: Review Request 53135: Fix constraint violations in adminprincipal table when installing database schema

2016-10-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53135/ --- (Updated Oct. 24, 2016, 10:30 a.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 53135: Fix constraint violations in adminprincipal table when installing database schema

2016-10-24 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53135/#review153687 --- Ship it! Ship It! - Nahappan Somasundaram On Oct. 24, 2016,

Review Request 53135: Fix constraint violations in adminprincipal table when installing database schema

2016-10-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53135/ --- Review request for Ambari, Jonathan Hurley, Laszlo Puskas, Nate Cole, and Oliver

Re: Review Request 53070: Add ability to configure sink configs for external AMS

2016-10-24 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53070/#review153686 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 20, 2016, 5:3

Re: Review Request 53101: While syncing with LDAP, username collisions should be handled based on configuration value

2016-10-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53101/#review153682 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Review Request 53129: AMBARI-18671 Ranger KMS should add proxy users for yarn and livy

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

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

2016-10-24 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52688/#review153680 --- Ship it! Ship It! - Sreenath Somarajapuram On Oct. 19, 2016,