Description
There are two issues with the current log configuration:
a) The currently used log pattern makes use of %C
{1}(classname) and %L (line numbers), which are, according to log4j docs, extremely slow. In addition, the information we get using these setting are quite useless due to b).
b) Instead of Log4J, commons logging finds the GeronimoLogFactory, which creates a wrapper for each log call. This result is that we lose the class and line number information of the originating log call and get instead something like GeronimoLog.debug(66).
Proposed fix:
a) ship ODE with a different default log pattern, including timestamp and log category (instead of classname and line numbers).
b) have an alternative log pattern for better debug output (classname, linenumbers, threadname).
c) set LogFactory to log4j.