Review Board 1.7.22


SQOOP-826 Sqoop2: Add log4j test configuration to all maven modules

Review Request #8922 - Created Jan. 10, 2013 and submitted

Jarek Cecho
SQOOP-826
Reviewers
Sqoop
sqoop-sqoop2
I've added the same configuration file to every maven module that have type "jar" or "war" (e.g. not to "dummy" modules of type "pom"). Please note that the "debug" setting should be fine as this file should be loaded only during tests.
I've run tests and I can no longer see the log4j warnings of missing configuration file.

Steps that I've used to verify this patch:

1) Run mvn clean test

2) grep "Please initialize the log4j system properly" * -r | wc -l

3) Apply my patch

4) Rerun steps 1 and 2, output should be "0"
Review request changed
Updated (Jan. 25, 2013, 2:29 a.m.)
  • changed from pending to submitted