> On April 23, 2014, 6:55 p.m., Mike Drob wrote: > > Since there is now (more) potential for divergent logging configuration, > > can you add an INFO level message on startup that identifies which > > configuration file is being used? If one alreadyd exists and I simple > > missed it, please disregard. > > Josh Elser wrote: > That was what I was getting at with -Dlog4j.debug which follows log4j > standards of printing out what file was used for configuring the framework. > > Bill Havanki wrote: > I lean toward relying on log4j.debug, in large part because if logging is > all messed up, then the INFO message saying what configuration file is in use > could get lost.
Yea, using log4j.debug is totally acceptable. - Mike ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20613/#review41187 ----------------------------------------------------------- On April 23, 2014, 4:52 p.m., Bill Havanki wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20613/ > ----------------------------------------------------------- > > (Updated April 23, 2014, 4:52 p.m.) > > > Review request for accumulo, Josh Elser and Vikram Srivastava. > > > Bugs: ACCUMULO-2383 > https://issues.apache.org/jira/browse/ACCUMULO-2383 > > > Repository: accumulo > > > Description > ------- > > 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. > > > Diffs > ----- > > conf/templates/generic_logger.properties PRE-CREATION > conf/templates/monitor_logger.properties PRE-CREATION > server/base/src/main/java/org/apache/accumulo/server/Accumulo.java 4e1eb35 > > server/base/src/main/java/org/apache/accumulo/server/watcher/MonitorLog4jWatcher.java > ac3426e > server/base/src/test/java/org/apache/accumulo/server/AccumuloTest.java > 9366163 > > server/base/src/test/java/org/apache/accumulo/server/watcher/MonitorLog4jWatcherTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/20613/diff/ > > > Testing > ------- > > - 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. > > > Thanks, > > Bill Havanki > >
