-----------------------------------------------------------
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 (updated)
-----

  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

Reply via email to