accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki" <>
Subject Review Request 20613: ACCUMULO-2383 - support for Log4j properties
Date Wed, 23 Apr 2014 16:53:05 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for accumulo, Josh Elser and Vikram Srivastava.

Bugs: ACCUMULO-2383

Repository: accumulo


Accumulo now looks for either XML or properties files for the Log4J configuration. Its MonitorLog4jWatcher
can now load either properties or XML.

This code depends on the v3 patch of ACCUMULO-2343; follow the "Depends On" link to jump to
that review.

Note on MonitorLog4jWatcher: Its constructor was calling setDelay() on its superclass, but
setDelay() is non-final. I took this opportunity to fix that.


  conf/templates/ PRE-CREATION 
  conf/templates/ PRE-CREATION 
  server/base/src/main/java/org/apache/accumulo/server/ 4e1eb35 
  server/base/src/main/java/org/apache/accumulo/server/watcher/ ac3426e

  server/base/src/test/java/org/apache/accumulo/server/ 9366163 



- New unit tests pass, such as they are.
- Ran servers with XML and with properties configurations on single-node cluster. Observed
reloading for properties files (didn't re-test XML). Checked that all logs were being generated.
Observed log messages sent from master and tserver to monitor, displayed on web.
- Ran short (50k-hop) randomwalk tests: Security, Concurrent, MultiTable.


Bill Havanki

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message