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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review155209 --- Pushed to trunk, commit 3ec7df13485b629bcf08310f3916b6401f827dbe

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review154706 --- What's the state of this review? Can it be closed? - Jonathan

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review153551 --- Ship it! Ship It! - Nate Cole On Oct. 19, 2016, 6:51 p.m.,

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

> On Oct. 18, 2016, 9:15 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 2428 > > > > > > nit: "Indicates the delay, in milliseconds, for

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/ --- (Updated Oct. 19, 2016, 10:51 p.m.) Review request for Ambari, Alejandro

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

> On Oct. 18, 2016, 7:37 a.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > lines 953-954 > > > > > > Variable names `monitor_delay` and

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review153176 --- Fix it, then Ship it!

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/ --- (Updated Oct. 18, 2016, 6:29 p.m.) Review request for Ambari, Alejandro

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

> On Oct. 18, 2016, 5:37 a.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > lines 950-951 > > > > > > Whitespace: missing between `){`, and

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

> 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

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review153034 ---

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

> 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

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

> 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

> 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

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

> 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

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

> On Oct. 14, 2016, 1:10 a.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

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review152643 ---

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

> 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

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

> On Oct. 13, 2016, 6:20 p.m., Alejandro Fernandez wrote: > > Ship It! thank you - Amarnath reddy --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review152549

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

> 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

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review152595 ---

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

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