Hi,
I've read some of the discussion on the irc channel as well
as the relevant page on confluence
(http://docs.codehaus.org/display/GEOSDEV/Commons-Logging+in+Geoserver)
and I have some feedback.
* On that wiki page I read:
"This CommonHandler class is configured by the Logging utility class (in the metadata module). To redirect a Logger to commons-logging, run the following code:
new Logging("org.geotools.data").redirectToCommonsLogging();"
Happily, there's no need to configure this on a package by package basis, we can do new Logging(Logging.ALL).redirectToCommonsLogging()
* I think we should limit ourselves to log4j configuration, if people
need to go java logging, they will do that by themselves. For java
logging, people can set environment variables to set his behaviour,
so I think it's still accessible from an administration point of view.
When the commons logging file points to something other than log4j, we
disable the user interface configuration and let's be done with it.
Life's too short to support multiple loggers, let's keep it simple.
* About configuring the logging level of the console from the user interface, it seems you're overriding what the config file contains,
right? Are we sure that's needed? I mean, having pre made configurations such as production/debug/fillMyHdWithLogs should be enough no?
Cheers
Andrea