----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43467/#review119407 -----------------------------------------------------------
test/src/test/resources/integration-tests-suite.xml (line 25) <https://reviews.apache.org/r/43467/#comment180762> How about add the listener for all suite like: connector-loading-tests-suite.xml and new-integration-tests-suite.xml, etc. - Colin Ma On Feb. 11, 2016, 9:29 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43467/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2016, 9:29 p.m.) > > > Review request for Sqoop and Jarek Cecho. > > > Bugs: SQOOP-2832 > https://issues.apache.org/jira/browse/SQOOP-2832 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > I feel that debugging anything in our pre-commit hook is currently mission > impossible from several reasons. One of the main ones is that we're > generating one single log file that has [grown to 1GB in > size|https://builds.apache.org/job/PreCommit-SQOOP-Build/2185/artifact/test/target/surefire-reports/] > with all "test related" logs and I'm not even remotely able to find anything > there. > > In normal case we would have one log per test class, but as we've refactored > our integration tests to run multiple classes inside single Suite, we're no > longer have that for free. We have done this change to cut the time it takes > to run the integration tests - before we were initializing mini clusters for > each class, which is extra ~40 seconds per class, so I don't think that > reverting it would be reasonable. > > We should perhaps explore other ways how to get multiple log files. > > > Diffs > ----- > > test/pom.xml 134bca1 > test/src/main/java/org/apache/sqoop/test/testng/ReconfigureLogListener.java > PRE-CREATION > test/src/test/resources/integration-tests-suite.xml 73e0a77 > > Diff: https://reviews.apache.org/r/43467/diff/ > > > Testing > ------- > > > Thanks, > > Jarek Cecho > >
