You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Jonathan Hurley <jh...@hortonworks.com> on 2015/08/09 04:20:17 UTC

Re: Review Request 37264: EclipseLink stdout Log Statements Fill Up Disk Space

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

(Updated Aug. 8, 2015, 10:20 p.m.)


Review request for Ambari, Alejandro Fernandez, Nate Cole, Sumit Mohanty, and Sid Wagle.


Bugs: AMBARI-12685
    https://issues.apache.org/jira/browse/AMBARI-12685


Repository: ambari


Description
-------

EclipseLink's default logger logs to standard output. Since Ambari redirects stdout to a file, this means that all EclipseLink statements are captured in {{ambari-server.out}}.

On some systems with latency problems, EclipseLink will log retry attempts. Since this log is not rolling, it can fill up the disk.

There is no log4j bridge for EclipseLink. This Jira will track the creation of an slf4j bridge that will log all SQL statements to a separate, rolling logger.


Diffs
-----

  ambari-server/conf/unix/log4j.properties f619c6c 
  ambari-server/conf/windows/log4j.properties 607cdf9 
  ambari-server/src/main/conf/log4j.properties 6d53ca3 
  ambari-server/src/main/java/org/apache/ambari/server/logging/EclipseLinkLogger.java PRE-CREATION 
  ambari-server/src/main/resources/META-INF/persistence.xml deb30bb 

Diff: https://reviews.apache.org/r/37264/diff/


Testing
-------

Verified new logger file created and is rolling. Examples of bridged log statements:

[EL Debug]: 2015-08-08 22:17:34.305--ServerSession(1653567754)--Connection(2013352132)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
	bind => [YARN, RESOURCEMANAGER, 2]
[EL Debug]: 2015-08-08 22:17:34.306--ServerSession(1653567754)--Connection(761406242)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
	bind => [ZOOKEEPER, ZOOKEEPER_SERVER, 2]
[EL Debug]: 2015-08-08 22:17:34.307--ServerSession(1653567754)--Connection(941036240)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE ((((service_name IN (?,?,?,?)) AND (scope = ?)) AND (cluster_id = ?)) AND (component_name IS NULL))
	bind => [ZOOKEEPER, MAPREDUCE2, HDFS, YARN, SERVICE, 2]


Thanks,

Jonathan Hurley


Re: Review Request 37264: EclipseLink stdout Log Statements Fill Up Disk Space

Posted by Sumit Mohanty <sm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37264/#review94734
-----------------------------------------------------------

Ship it!


Ship It!

- Sumit Mohanty


On Aug. 10, 2015, 12:46 p.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37264/
> -----------------------------------------------------------
> 
> (Updated Aug. 10, 2015, 12:46 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Nate Cole, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-12685
>     https://issues.apache.org/jira/browse/AMBARI-12685
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> EclipseLink's default logger logs to standard output. Since Ambari redirects stdout to a file, this means that all EclipseLink statements are captured in {{ambari-server.out}}.
> 
> On some systems with latency problems, EclipseLink will log retry attempts. Since this log is not rolling, it can fill up the disk.
> 
> There is no log4j bridge for EclipseLink. This Jira will track the creation of an slf4j bridge that will log all SQL statements to a separate, rolling logger.
> 
> 
> Diffs
> -----
> 
>   ambari-server/conf/unix/log4j.properties f619c6c 
>   ambari-server/conf/windows/log4j.properties 607cdf9 
>   ambari-server/src/main/conf/log4j.properties 6d53ca3 
>   ambari-server/src/main/java/org/apache/ambari/server/logging/EclipseLinkLogger.java PRE-CREATION 
>   ambari-server/src/main/resources/META-INF/persistence.xml deb30bb 
> 
> Diff: https://reviews.apache.org/r/37264/diff/
> 
> 
> Testing
> -------
> 
> Verified new logger file created and is rolling. Examples of bridged log statements:
> 
> [EL Debug]: 2015-08-08 22:17:34.305--ServerSession(1653567754)--Connection(2013352132)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [YARN, RESOURCEMANAGER, 2]
> [EL Debug]: 2015-08-08 22:17:34.306--ServerSession(1653567754)--Connection(761406242)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [ZOOKEEPER, ZOOKEEPER_SERVER, 2]
> [EL Debug]: 2015-08-08 22:17:34.307--ServerSession(1653567754)--Connection(941036240)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE ((((service_name IN (?,?,?,?)) AND (scope = ?)) AND (cluster_id = ?)) AND (component_name IS NULL))
> 	bind => [ZOOKEEPER, MAPREDUCE2, HDFS, YARN, SERVICE, 2]
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>


Re: Review Request 37264: EclipseLink stdout Log Statements Fill Up Disk Space

Posted by Nate Cole <nc...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37264/#review94741
-----------------------------------------------------------

Ship it!


Ship It!

- Nate Cole


On Aug. 10, 2015, 8:46 a.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37264/
> -----------------------------------------------------------
> 
> (Updated Aug. 10, 2015, 8:46 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Nate Cole, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-12685
>     https://issues.apache.org/jira/browse/AMBARI-12685
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> EclipseLink's default logger logs to standard output. Since Ambari redirects stdout to a file, this means that all EclipseLink statements are captured in {{ambari-server.out}}.
> 
> On some systems with latency problems, EclipseLink will log retry attempts. Since this log is not rolling, it can fill up the disk.
> 
> There is no log4j bridge for EclipseLink. This Jira will track the creation of an slf4j bridge that will log all SQL statements to a separate, rolling logger.
> 
> 
> Diffs
> -----
> 
>   ambari-server/conf/unix/log4j.properties f619c6c 
>   ambari-server/conf/windows/log4j.properties 607cdf9 
>   ambari-server/src/main/conf/log4j.properties 6d53ca3 
>   ambari-server/src/main/java/org/apache/ambari/server/logging/EclipseLinkLogger.java PRE-CREATION 
>   ambari-server/src/main/resources/META-INF/persistence.xml deb30bb 
> 
> Diff: https://reviews.apache.org/r/37264/diff/
> 
> 
> Testing
> -------
> 
> Verified new logger file created and is rolling. Examples of bridged log statements:
> 
> [EL Debug]: 2015-08-08 22:17:34.305--ServerSession(1653567754)--Connection(2013352132)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [YARN, RESOURCEMANAGER, 2]
> [EL Debug]: 2015-08-08 22:17:34.306--ServerSession(1653567754)--Connection(761406242)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [ZOOKEEPER, ZOOKEEPER_SERVER, 2]
> [EL Debug]: 2015-08-08 22:17:34.307--ServerSession(1653567754)--Connection(941036240)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE ((((service_name IN (?,?,?,?)) AND (scope = ?)) AND (cluster_id = ?)) AND (component_name IS NULL))
> 	bind => [ZOOKEEPER, MAPREDUCE2, HDFS, YARN, SERVICE, 2]
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>


Re: Review Request 37264: EclipseLink stdout Log Statements Fill Up Disk Space

Posted by Jonathan Hurley <jh...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37264/
-----------------------------------------------------------

(Updated Aug. 10, 2015, 8:46 a.m.)


Review request for Ambari, Alejandro Fernandez, Nate Cole, Sumit Mohanty, and Sid Wagle.


Bugs: AMBARI-12685
    https://issues.apache.org/jira/browse/AMBARI-12685


Repository: ambari


Description
-------

EclipseLink's default logger logs to standard output. Since Ambari redirects stdout to a file, this means that all EclipseLink statements are captured in {{ambari-server.out}}.

On some systems with latency problems, EclipseLink will log retry attempts. Since this log is not rolling, it can fill up the disk.

There is no log4j bridge for EclipseLink. This Jira will track the creation of an slf4j bridge that will log all SQL statements to a separate, rolling logger.


Diffs (updated)
-----

  ambari-server/conf/unix/log4j.properties f619c6c 
  ambari-server/conf/windows/log4j.properties 607cdf9 
  ambari-server/src/main/conf/log4j.properties 6d53ca3 
  ambari-server/src/main/java/org/apache/ambari/server/logging/EclipseLinkLogger.java PRE-CREATION 
  ambari-server/src/main/resources/META-INF/persistence.xml deb30bb 

Diff: https://reviews.apache.org/r/37264/diff/


Testing
-------

Verified new logger file created and is rolling. Examples of bridged log statements:

[EL Debug]: 2015-08-08 22:17:34.305--ServerSession(1653567754)--Connection(2013352132)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
	bind => [YARN, RESOURCEMANAGER, 2]
[EL Debug]: 2015-08-08 22:17:34.306--ServerSession(1653567754)--Connection(761406242)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
	bind => [ZOOKEEPER, ZOOKEEPER_SERVER, 2]
[EL Debug]: 2015-08-08 22:17:34.307--ServerSession(1653567754)--Connection(941036240)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE ((((service_name IN (?,?,?,?)) AND (scope = ?)) AND (cluster_id = ?)) AND (component_name IS NULL))
	bind => [ZOOKEEPER, MAPREDUCE2, HDFS, YARN, SERVICE, 2]


Thanks,

Jonathan Hurley


Re: Review Request 37264: EclipseLink stdout Log Statements Fill Up Disk Space

Posted by Sumit Mohanty <sm...@hortonworks.com>.

> On Aug. 9, 2015, 3:28 p.m., Sumit Mohanty wrote:
> > ambari-server/conf/unix/log4j.properties, line 56
> > <https://reviews.apache.org/r/37264/diff/1/?file=1035325#file1035325line56>
> >
> >     Should this be RollingFileAppender?
> 
> Jonathan Hurley wrote:
>     Thanks for the review. 
>     
>     Probably - I debated this since log4j.properties is customer-configurable. I wasn't sure if making it rolling out of the gate would be undesireable. I think it probably should be though.

Agree. At least the driving scenario for the change, slower off-node connection to SQL, will benefit from it being rolling.


- Sumit


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


On Aug. 10, 2015, 12:46 p.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37264/
> -----------------------------------------------------------
> 
> (Updated Aug. 10, 2015, 12:46 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Nate Cole, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-12685
>     https://issues.apache.org/jira/browse/AMBARI-12685
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> EclipseLink's default logger logs to standard output. Since Ambari redirects stdout to a file, this means that all EclipseLink statements are captured in {{ambari-server.out}}.
> 
> On some systems with latency problems, EclipseLink will log retry attempts. Since this log is not rolling, it can fill up the disk.
> 
> There is no log4j bridge for EclipseLink. This Jira will track the creation of an slf4j bridge that will log all SQL statements to a separate, rolling logger.
> 
> 
> Diffs
> -----
> 
>   ambari-server/conf/unix/log4j.properties f619c6c 
>   ambari-server/conf/windows/log4j.properties 607cdf9 
>   ambari-server/src/main/conf/log4j.properties 6d53ca3 
>   ambari-server/src/main/java/org/apache/ambari/server/logging/EclipseLinkLogger.java PRE-CREATION 
>   ambari-server/src/main/resources/META-INF/persistence.xml deb30bb 
> 
> Diff: https://reviews.apache.org/r/37264/diff/
> 
> 
> Testing
> -------
> 
> Verified new logger file created and is rolling. Examples of bridged log statements:
> 
> [EL Debug]: 2015-08-08 22:17:34.305--ServerSession(1653567754)--Connection(2013352132)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [YARN, RESOURCEMANAGER, 2]
> [EL Debug]: 2015-08-08 22:17:34.306--ServerSession(1653567754)--Connection(761406242)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [ZOOKEEPER, ZOOKEEPER_SERVER, 2]
> [EL Debug]: 2015-08-08 22:17:34.307--ServerSession(1653567754)--Connection(941036240)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE ((((service_name IN (?,?,?,?)) AND (scope = ?)) AND (cluster_id = ?)) AND (component_name IS NULL))
> 	bind => [ZOOKEEPER, MAPREDUCE2, HDFS, YARN, SERVICE, 2]
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>


Re: Review Request 37264: EclipseLink stdout Log Statements Fill Up Disk Space

Posted by Jonathan Hurley <jh...@hortonworks.com>.

> On Aug. 9, 2015, 11:28 a.m., Sumit Mohanty wrote:
> > ambari-server/conf/unix/log4j.properties, line 56
> > <https://reviews.apache.org/r/37264/diff/1/?file=1035325#file1035325line56>
> >
> >     Should this be RollingFileAppender?

Thanks for the review. 

Probably - I debated this since log4j.properties is customer-configurable. I wasn't sure if making it rolling out of the gate would be undesireable. I think it probably should be though.


- Jonathan


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


On Aug. 8, 2015, 10:20 p.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37264/
> -----------------------------------------------------------
> 
> (Updated Aug. 8, 2015, 10:20 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Nate Cole, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-12685
>     https://issues.apache.org/jira/browse/AMBARI-12685
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> EclipseLink's default logger logs to standard output. Since Ambari redirects stdout to a file, this means that all EclipseLink statements are captured in {{ambari-server.out}}.
> 
> On some systems with latency problems, EclipseLink will log retry attempts. Since this log is not rolling, it can fill up the disk.
> 
> There is no log4j bridge for EclipseLink. This Jira will track the creation of an slf4j bridge that will log all SQL statements to a separate, rolling logger.
> 
> 
> Diffs
> -----
> 
>   ambari-server/conf/unix/log4j.properties f619c6c 
>   ambari-server/conf/windows/log4j.properties 607cdf9 
>   ambari-server/src/main/conf/log4j.properties 6d53ca3 
>   ambari-server/src/main/java/org/apache/ambari/server/logging/EclipseLinkLogger.java PRE-CREATION 
>   ambari-server/src/main/resources/META-INF/persistence.xml deb30bb 
> 
> Diff: https://reviews.apache.org/r/37264/diff/
> 
> 
> Testing
> -------
> 
> Verified new logger file created and is rolling. Examples of bridged log statements:
> 
> [EL Debug]: 2015-08-08 22:17:34.305--ServerSession(1653567754)--Connection(2013352132)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [YARN, RESOURCEMANAGER, 2]
> [EL Debug]: 2015-08-08 22:17:34.306--ServerSession(1653567754)--Connection(761406242)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [ZOOKEEPER, ZOOKEEPER_SERVER, 2]
> [EL Debug]: 2015-08-08 22:17:34.307--ServerSession(1653567754)--Connection(941036240)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE ((((service_name IN (?,?,?,?)) AND (scope = ?)) AND (cluster_id = ?)) AND (component_name IS NULL))
> 	bind => [ZOOKEEPER, MAPREDUCE2, HDFS, YARN, SERVICE, 2]
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>


Re: Review Request 37264: EclipseLink stdout Log Statements Fill Up Disk Space

Posted by Sumit Mohanty <sm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37264/#review94676
-----------------------------------------------------------



ambari-server/conf/unix/log4j.properties (line 56)
<https://reviews.apache.org/r/37264/#comment149281>

    Should this be RollingFileAppender?


- Sumit Mohanty


On Aug. 9, 2015, 2:20 a.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37264/
> -----------------------------------------------------------
> 
> (Updated Aug. 9, 2015, 2:20 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Nate Cole, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-12685
>     https://issues.apache.org/jira/browse/AMBARI-12685
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> EclipseLink's default logger logs to standard output. Since Ambari redirects stdout to a file, this means that all EclipseLink statements are captured in {{ambari-server.out}}.
> 
> On some systems with latency problems, EclipseLink will log retry attempts. Since this log is not rolling, it can fill up the disk.
> 
> There is no log4j bridge for EclipseLink. This Jira will track the creation of an slf4j bridge that will log all SQL statements to a separate, rolling logger.
> 
> 
> Diffs
> -----
> 
>   ambari-server/conf/unix/log4j.properties f619c6c 
>   ambari-server/conf/windows/log4j.properties 607cdf9 
>   ambari-server/src/main/conf/log4j.properties 6d53ca3 
>   ambari-server/src/main/java/org/apache/ambari/server/logging/EclipseLinkLogger.java PRE-CREATION 
>   ambari-server/src/main/resources/META-INF/persistence.xml deb30bb 
> 
> Diff: https://reviews.apache.org/r/37264/diff/
> 
> 
> Testing
> -------
> 
> Verified new logger file created and is rolling. Examples of bridged log statements:
> 
> [EL Debug]: 2015-08-08 22:17:34.305--ServerSession(1653567754)--Connection(2013352132)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [YARN, RESOURCEMANAGER, 2]
> [EL Debug]: 2015-08-08 22:17:34.306--ServerSession(1653567754)--Connection(761406242)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE (((service_name = ?) AND (component_name = ?)) AND (cluster_id = ?))
> 	bind => [ZOOKEEPER, ZOOKEEPER_SERVER, 2]
> [EL Debug]: 2015-08-08 22:17:34.307--ServerSession(1653567754)--Connection(941036240)--Thread(Thread[qtp-ambari-agent-57,5,main])-- SELECT definition_id, cluster_id, component_name, definition_name, description, ENABLED, HASH, ignore_host, label, schedule_interval, scope, service_name, alert_source, source_type FROM alert_definition WHERE ((((service_name IN (?,?,?,?)) AND (scope = ?)) AND (cluster_id = ?)) AND (component_name IS NULL))
> 	bind => [ZOOKEEPER, MAPREDUCE2, HDFS, YARN, SERVICE, 2]
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>