I've see this pattern:
LOGGER.error("Error parsing {}", configSource.getLocation(), e);
Which hides the exception. I changed it to:
LOGGER.error("Error parsing " + configSource.getLocation(), e);
I think we need something like:
error(Throwable, String, Object...)
Gary
--
E-Mail: [email protected] | [email protected]
Java Persistence with Hibernate, Second Edition
<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory
