Re: Review Request 49029: When requesting a Kerberos Descriptor via the REST API, 'when' clauses should optionally be processed

2016-06-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49029/#review139080 --- Ship it!

Re: Review Request 49087: After registering 2.5 HDP repo, it is hidden on UI

2016-06-22 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49087/ --- (Updated June 22, 2016, 5:55 p.m.) Review request for Ambari, Dmytro Grinenko,

Review Request 49092: Zeppelin interpreter config typo

2016-06-22 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49092/ --- Review request for Ambari, DIPAYAN BHOWMICK, Pallav Kulshreshtha, and Rohit

Review Request 49093: Unable To Delete A Service After Upgrade Due To Foreign Key Violation

2016-06-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49093/ --- Review request for Ambari, Alejandro Fernandez and Robert Levas. Bugs:

Re: Review Request 49086: hiveserver2-site.xml not created

2016-06-22 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49086/#review139066 --- Ship it! Ship It! - Robert Levas On June 22, 2016, 9:41

Re: Review Request 49029: When requesting a Kerberos Descriptor via the REST API, 'when' clauses should optionally be processed

2016-06-22 Thread Robert Levas
> On June 22, 2016, 12:43 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterKerberosDescriptorResourceProvider.java, > > line 335 > > > > > > isBlank

Re: Review Request 49029: When requesting a Kerberos Descriptor via the REST API, 'when' clauses should optionally be processed

2016-06-22 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49029/ --- (Updated June 22, 2016, 1:57 p.m.) Review request for Ambari, Di Li, Jonathan

Re: Review Request 48950: Pig View- Show actual error message thrown by webhcat api

2016-06-22 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48950/ --- (Updated June 22, 2016, 7:13 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 48950: Pig View- Show actual error message thrown by webhcat api

2016-06-22 Thread Nitiraj Rathore
> On June 21, 2016, 7:12 a.m., Nitiraj Rathore wrote: > > ambari-server/src/main/java/org/apache/ambari/server/view/ViewAmbariStreamProvider.java, > > line 123 > > > > > > is this path being passed just for logging

Re: Review Request 49012: AMBARI-17330 Ambari changes to support kerberized Ranger tagsync

2016-06-22 Thread Mugdha Varadkar
> On June 21, 2016, 2:31 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.6.0/kerberos.json, > > lines 114-122 > > > > > > This should go into a Jinja2 template file, which is

Re: Review Request 49012: AMBARI-17330 Ambari changes to support kerberized Ranger tagsync

2016-06-22 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49012/ --- (Updated June 22, 2016, 6:30 a.m.) Review request for Ambari, Gautam Borad,

Re: Review Request 48395: AMBARI-17027: Metrics Collector API: Introduce basic series aggregation functions

2016-06-22 Thread Prajwal Rao
> On June 22, 2016, 6:32 a.m., Prajwal Rao wrote: > > Ship It! Datasource changes look good to me. - Prajwal --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48395/#review139022

Re: Review Request 48395: AMBARI-17027: Metrics Collector API: Introduce basic series aggregation functions

2016-06-22 Thread Prajwal Rao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48395/#review139022 --- Ship it! Ship It! - Prajwal Rao On June 22, 2016, 2:09

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

2016-06-22 Thread Sreenath Somarajapuram
> On June 22, 2016, 8:50 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js, > > line 164 > > > > > > Why are we having the + after ? > > Akhil PB

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

2016-06-22 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48487/#review139036 ---

Re: Review Request 49028: Alert mapreduce_history_server_process fails after enabling SSL

2016-06-22 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49028/#review139034 --- Ship it! Ship It! - Dmitro Lisnichenko On June 21, 2016,

Re: Review Request 49077: Could not download Atlas Client configs

2016-06-22 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49077/#review139039 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 22, 2016,

Review Request 49077: Could not download Atlas Client configs

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

Re: Review Request 49032: Hive metastore failed to start

2016-06-22 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49032/#review139035 --- Ship it! Ship It! - Dmitro Lisnichenko On June 21, 2016,

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

2016-06-22 Thread Akhil PB
> On June 22, 2016, 8:50 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js, > > line 164 > > > > > > Why are we having the + after ? Converting

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

2016-06-22 Thread Akhil PB
> On June 22, 2016, 8:50 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js, > > line 164 > > > > > > Why are we having the + after ? > > Akhil PB

Review Request 49078: AMBARI-17363: Enable Ranger support for Hive Interactive

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

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

2016-06-22 Thread Akhil PB
> On June 22, 2016, 10:12 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/versionsPanel.hbs, > > line 26 > > > > > > Where is the versions-table-wrap

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

2016-06-22 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48487/#review139032 ---

Re: Review Request 49080: Tez install failed

2016-06-22 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49080/ --- (Updated Червень 22, 2016, 11:58 до полудня) Review request for Ambari, Andrew

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

2016-06-22 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48487/#review139046 ---

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

2016-06-22 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48487/#review139051 --- - Sreenath Somarajapuram On June 9, 2016, 12:05 p.m., Akhil PB

Re: Review Request 48722: Reduce the idle time before first command from next stage is executed on a host

2016-06-22 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48722/#review139054 --- Ship it! Ship It! - Andrew Onischuk On June 22, 2016, 11:21

Re: Review Request 49081: Ambari-on-LLAP: MetadataSpace memory setting

2016-06-22 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49081/#review139056 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 22, 2016,

Re: Review Request 49080: Tez install failed

2016-06-22 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49080/#review139052 --- Ship it! Ship It! - Dmytro Sen On Июнь 22, 2016, 11:12

Re: Review Request 48722: Reduce the idle time before first command from next stage is executed on a host

2016-06-22 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48722/ --- (Updated June 22, 2016, 1:10 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 48722: Reduce the idle time before first command from next stage is executed on a host

2016-06-22 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48722/ --- (Updated June 22, 2016, 1:21 p.m.) Review request for Ambari, Andrew Onischuk,

Review Request 49081: Ambari-on-LLAP: MetadataSpace memory setting

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

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

2016-06-22 Thread Sreenath Somarajapuram
> On June 22, 2016, 10:12 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/versionsPanel.hbs, > > line 26 > > > > > > Where is the versions-table-wrap

Review Request 49080: Tez install failed

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

Re: Review Request 48722: Reduce the idle time before first command from next stage is executed on a host

2016-06-22 Thread Victor Galgo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48722/#review139053 --- Ship it! Ship It! - Victor Galgo On June 22, 2016, 11:21

Review Request 49086: hiveserver2-site.xml not created

2016-06-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49086/ --- Review request for Ambari, Alejandro Fernandez, Robert Levas, and Sumit Mohanty.

Review Request 49087: After registering 2.5 HDP repo, it is hidden on UI

2016-06-22 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49087/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-17370

Re: Review Request 49087: After registering 2.5 HDP repo, it is hidden on UI

2016-06-22 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49087/ --- (Updated June 22, 2016, 4:45 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 49029: When requesting a Kerberos Descriptor via the REST API, 'when' clauses should optionally be processed

2016-06-22 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49029/ --- (Updated June 22, 2016, 10:17 a.m.) Review request for Ambari, Di Li, Jonathan

Re: Review Request 48950: Pig View- Show actual error message thrown by webhcat api

2016-06-22 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48950/#review139060 --- Ship it! Ship It! - Nitiraj Rathore On June 22, 2016, 7:13

Re: Review Request 49072: AMBARI-17333 ranger kms repo creation is failing after ranger kms is installed

2016-06-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49072/#review139062 --- Can we get some unit tests to cover this case? - Jonathan Hurley

Re: Review Request 48507: AMBARI-17074: Expose Spark daemon memory in Spark2

2016-06-22 Thread Weiqing Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48507/ --- (Updated June 22, 2016, 6:09 p.m.) Review request for Ambari, Zhe (Joe) Wang,

Re: Review Request 49093: Unable To Delete A Service After Upgrade Due To Foreign Key Violation

2016-06-22 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49093/#review139074 --- Ship it! Ship It! - Robert Levas On June 22, 2016, 11:35

Re: Review Request 49087: After registering 2.5 HDP repo, it is hidden on UI

2016-06-22 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49087/#review139076 --- Ship it! Ship It! - Dmytro Grinenko On June 22, 2016, 4:09

Review Request 49119: AMBARI-17383: User names should be case insensitive

2016-06-22 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49119/ --- Review request for Ambari, Myroslav Papirkovskyy, Robert Levas, and Sumit

Review Request 49106: AuthorizationHelperTest.testAuthName may fail due to residual SecurityContext from previously executed test cases.

2016-06-22 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49106/ --- Review request for Ambari, Aravindan Vijayan, Jonathan Hurley, and Yusaku Sako.

Re: Review Request 49104: The DataNode Unmounted Alert Produces False Alerts When file:// URIs Are Used

2016-06-22 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49104/#review139139 --- Ship it! Ship It! - Alejandro Fernandez On June 22, 2016,

Re: Review Request 49114: AMBARI-17384. Remove '/usr/hdp' hardcoding from HIVE's alert_llap_app_status.py and KNOX's upgrade.py

2016-06-22 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49114/#review139162 --- Ship it! Ship It! - Sumit Mohanty On June 22, 2016, 10:51

Review Request 49124: AMBARI-17387 Changing HDP repo URL does not leads to sending written URL to ambari-server while registering new HDP version

2016-06-22 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49124/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Xi Wang, and Yusaku Sako.

Re: Review Request 49114: AMBARI-17384. Remove '/usr/hdp' hardcoding from HIVE's alert_llap_app_status.py and KNOX's upgrade.py

2016-06-22 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49114/#review139167 --- Ship it! Ship It! - Alejandro Fernandez On June 22, 2016,

Review Request 49104: The DataNode Unmounted Alert Produces False Alerts When file:// URIs Are Used

2016-06-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49104/ --- Review request for Ambari, Alejandro Fernandez and Robert Levas. Bugs:

Re: Review Request 49104: The DataNode Unmounted Alert Produces False Alerts When file:// URIs Are Used

2016-06-22 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49104/#review139142 --- Ship it! Ship It! - Robert Levas On June 22, 2016, 5:25

Re: Review Request 49106: AuthorizationHelperTest.testAuthName may fail due to residual SecurityContext from previously executed test cases.

2016-06-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49106/#review139140 --- Ship it! Ship It! - Jonathan Hurley On June 22, 2016, 5:22

Review Request 49114: AMBARI-17384. Remove '/usr/hdp' hardcoding from HIVE's alert_llap_app_status.py and KNOX's upgrade.py

2016-06-22 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49114/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 49104: The DataNode Unmounted Alert Produces False Alerts When file:// URIs Are Used

2016-06-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49104/ --- (Updated June 22, 2016, 5:25 p.m.) Review request for Ambari, Alejandro

Review Request 49121: AMBARI-17386. Misc fixes for Atlas, write correct conf files for server and client, use PLAINTEXTSASL, and remove atlas from HADOOP_CLASSPATH in Hive

2016-06-22 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49121/ --- Review request for Ambari, Oliver Szabo, Sumit Mohanty, and Suma Shivaprasad.

Review Request 49128: AMBARI-17389. Read 'yarn.nodemanager.resource.memory-mb' and 'yarn.scheduler.minimum-allocation-mb' from 'configurations' if 'changed-configurations' is empty and config is there

2016-06-22 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49128/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 49128: AMBARI-17389. Read 'yarn.nodemanager.resource.memory-mb' and 'yarn.scheduler.minimum-allocation-mb' from 'configurations' if 'changed-configurations' is empty and config is t

2016-06-22 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49128/#review139206 --- Ship it! Ship It! - Sumit Mohanty On June 23, 2016, 2:28

Review Request 49126: AMBARI-17388 - Add Service Wizard stuck on Customize Services step

2016-06-22 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49126/ --- Review request for Ambari and Zhe (Joe) Wang. Bugs: AMBARI-17388

Re: Review Request 49126: AMBARI-17388 - Add Service Wizard stuck on Customize Services step

2016-06-22 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49126/#review139192 --- Ship it! Ship It! - Zhe (Joe) Wang On June 23, 2016, 12:34

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

2016-06-22 Thread Masahiro Tanaka
> On 6月 21, 2016, 3:05 p.m., Jonathan Hurley wrote: > > ambari-agent/src/main/python/ambari_agent/alerts/port_alert.py, lines > > 133-135 > > > > > > We don't hardcode specific workarounds into the alerts

Re: Review Request 49012: AMBARI-17330 Ambari changes to support kerberized Ranger tagsync

2016-06-22 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49012/#review139120 --- Ship it! Ship It! - Robert Levas On June 22, 2016, 2:30