-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20465/
-----------------------------------------------------------
(Updated April 18, 2014, 3:14 p.m.)
Review request for accumulo and Josh Elser.
Changes
-------
Add MonitorLoggingIT functional test, which starts a minicluster with monitor
and checks that some log events show up on the monitor's log page. The
AsyncSocketAppender.main() method is removed.
Bugs: ACCUMULO-2343
https://issues.apache.org/jira/browse/ACCUMULO-2343
Repository: accumulo
Description
-------
AsyncSocketAppender is a Log4J AsyncAppender with its own internal
SocketAppender. Configuration for either appender can be set on the
AsyncSocketAppender itself. An AsyncSocketAppender can be configured using a
Log4J properties file, while an ordinary AsyncAppender cannot.
Diffs (updated)
-----
core/src/main/java/org/apache/accumulo/core/util/AsyncSocketAppender.java
PRE-CREATION
core/src/test/java/org/apache/accumulo/core/util/AsyncSocketAppenderTest.java
PRE-CREATION
test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java
d9bed7f
test/src/test/java/org/apache/accumulo/test/functional/MonitorLoggingIT.java
PRE-CREATION
test/src/test/resources/conf/generic_logger.xml PRE-CREATION
test/src/test/resources/conf/monitor_logger.xml PRE-CREATION
Diff: https://reviews.apache.org/r/20465/diff/
Testing
-------
Unit test created and passed. Also used main() method to send log messages to a
running Log4J SimpleSocketServer instance.
Thanks,
Bill Havanki
