Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-17 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review153034 --- ambari-server/src/main/java/org/apache/ambari/server/controller/A

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-17 Thread Attila Doroszlai
> On Oct. 14, 2016, 10:59 a.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 2430 > > > > > > How about using `TimeUnit.MINUTES.toMilli

Re: Review Request 52971: Service Summary screen does not show hostname in tooltip pop-up for all services

2016-10-17 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52971/#review153015 --- Ship it! Ship It! - Yusaku Sako On Oct. 18, 2016, 1:39 a.m.,

Review Request 52971: Service Summary screen does not show hostname in tooltip pop-up for all services

2016-10-17 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52971/ --- Review request for Ambari, Mahadev Konar, Sumit Mohanty, and Yusaku Sako. Bugs:

Re: Review Request 52964: Optimize Service Checks to it picks a random host and prefers hosts with 0 active commands

2016-10-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52964/#review153002 --- ambari-server/src/main/java/org/apache/ambari/server/controller/A

Review Request 52964: Optimize Service Checks to it picks a random host and prefers hosts with 0 active commands

2016-10-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52964/ --- Review request for Ambari, Dmytro Grinenko, Di Li, Dmitro Lisnichenko, Jonathan

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-17 Thread Amarnath reddy pappu
> On Oct. 13, 2016, 7:35 p.m., Jonathan Hurley wrote: > > Ship It! thank you - Amarnath reddy --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review152562 -

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-17 Thread Amarnath reddy pappu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/ --- (Updated Oct. 17, 2016, 11:07 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-17 Thread Amarnath reddy pappu
> On Oct. 13, 2016, 11:10 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > line 512 > > > > > > I'm curious - why use the configuration property getDe

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-17 Thread Amarnath reddy pappu
> On Oct. 14, 2016, 8:59 a.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 2430 > > > > > > How about using `TimeUnit.MINUTES.toMillis

Review Request 52958: Fix Log Search User Config bug

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

Re: Review Request 49838: AMBARI-17636 Service Configs page: can't see all config versions in dropdown

2016-10-17 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49838/ --- (Updated Oct. 17, 2016, 9:50 p.m.) Review request for Ambari, Aleksandr Kovalen

Re: Review Request 52691: Provision actions to happen based only on specified dependencies

2016-10-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52691/#review152962 --- ambari-server/src/main/java/org/apache/ambari/server/actionmanage

Re: Review Request 52850: Icons don't display in Windows 10 Microsoft Edge or Internet Explorer 11

2016-10-17 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52850/#review152960 --- Ship it! Ship It! - Zhe (Joe) Wang On Oct. 14, 2016, 12:07 a

Re: Review Request 49838: AMBARI-17636 Service Configs page: can't see all config versions in dropdown

2016-10-17 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49838/#review152958 --- Ship it! Ship It! - Zhe (Joe) Wang On Oct. 13, 2016, 12:35 a

Re: Review Request 52932: Ambari MIB Should Be Included with Installation Bits

2016-10-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52932/#review152957 --- ambari-server/pom.xml (line 44)

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-10-17 Thread Jonathan Hurley
> On Oct. 13, 2016, 7:10 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > line 512 > > > > > > I'm curious - why use the configuration property getDef

Re: Review Request 52369: AMBARI-12263: Support PAM as authentication mechanism for accessing Ambari UI/REST

2016-10-17 Thread Vishal Ghugare
> On Oct. 17, 2016, 8:09 a.m., Robert Levas wrote: > > It seems like there was a rebase issue with the last patch. Can you fix? fixed. please ignore revision 5. - Vishal --- This is an automatically generated e-mail. To reply, visit: ht

Re: Review Request 52369: AMBARI-12263: Support PAM as authentication mechanism for accessing Ambari UI/REST

2016-10-17 Thread Vishal Ghugare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52369/ --- (Updated Oct. 17, 2016, 1:50 p.m.) Review request for Ambari, Alejandro Fernand

Review Request 52932: Ambari MIB Should Be Included with Installation Bits

2016-10-17 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52932/ --- Review request for Ambari, Aravindan Vijayan, Jonathan Hurley, and Sid Wagle. B

Re: Review Request 52917: Improve Audit Log processing by Logfeeder

2016-10-17 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52917/#review152885 --- Ship it! Ship It! - Robert Nettleton On Oct. 15, 2016, 11:48

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

2016-10-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52930/#review152883 --- Fix it, then Ship it! Ship It! ambari-metrics/ambari-metrics-

Re: Review Request 52369: AMBARI-12263: Support PAM as authentication mechanism for accessing Ambari UI/REST

2016-10-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52369/#review152880 --- It seems like there was a rebase issue with the last patch. Can yo

Re: Review Request 49484: Remove spark.yarn.max.executor.failures configuration in Spark Ambari definition

2016-10-17 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49484/#review152878 --- Ship it! Ship It! - Sumit Mohanty On Oct. 17, 2016, 6:40 a.m

Re: Review Request 52691: Provision actions to happen based only on specified dependencies

2016-10-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52691/#review152873 --- ambari-server/src/main/java/org/apache/ambari/server/actionmanage

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

2016-10-17 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52930/ --- Review request for Ambari, Aravindan Vijayan, Robert Levas, and Sid Wagle. Bugs

Re: Review Request 52691: Provision actions to happen based only on specified dependencies

2016-10-17 Thread Nate Cole
> On Oct. 14, 2016, 10:40 a.m., Sebastian Toader wrote: > > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql, line 359 > > > > > > This could be an 'int' column and the values stored in it mapped to he >

Re: Review Request 52691: Provision actions to happen based only on specified dependencies

2016-10-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52691/#review152841 --- Fix it, then Ship it! Has this been tested using both models wh

Re: Review Request 52928: Give a reasonable error message when oozie service_check fails due to nodocs docker option

2016-10-17 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52928/#review152864 --- Ship it! Ship It! - Dmitro Lisnichenko On Oct. 17, 2016, 3:3

Review Request 52928: Give a reasonable error message when oozie service_check fails due to nodocs docker option

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

Re: Review Request 52691: Provision actions to happen based only on specified dependencies

2016-10-17 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52691/#review152833 --- Ship it! Ship It! - Sebastian Toader On Oct. 16, 2016, 10:33