Good point, I'll address that.

Because I felt that it's redundant information - the %c{2} will print classname and the package name immediately left of classname. However the [%1] will print full class name including all packages. Check out the current log line:

2015-10-05 10:14:02,197 DEBUG common.CommonRepositoryHandler [org.apache.sqoop.repository.common.CommonRepositoryHandler.findDriver(CommonRepositoryHandler.java:325)] Looking up Driver and config

The fragment common.CommonRepositoryHandler is there twice.

I'm not concerned about that - without INFO most of the log is pretty much useless as we don't know what's going on. Originally Sqoop classes were configured for INFO as well and the warn was only for "other" classes. As we're not running much other classes inside Sqoop, I'm not anticipating that this change will have huge impact.
- Jarek
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38942/#review101486
-----------------------------------------------------------
On Oct. 1, 2015, 11:32 p.m., Jarek Cecho wrote:

NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB