GitHub user ilooner opened a pull request:
https://github.com/apache/drill/pull/1053
DRILL-5989 Travis Finally Runs Smoke Tests!!!
## DRILL-5989
Previously Travis would only do a build for each pull request. Now Travis
will also run all the smoke tests every time a PR is opened or updated. Smoke
Tests are defined as tests which do not belong to one of these categories:
- **org.apache.drill.categories.SlowTest**
- **org.apache.drill.categories.SecurityTest**
- **org.apache.drill.categories.UnlikelyTest**
In order to get the smoke tests to run on Travis I had to make the
following fixes:
- Categorized Kafka storage plugin tests
- Categorized more long running tests as slow tests in order to avoid
timing out on travis
- Fixed a bug with jdbc-all tests which would cause jdbc-all to fail when
categories were excluded
- Fixed usage of surefire plugin in java-exec, and fixed a bug which caused
**excludedGroups** not to be propagated to the java-exec project.
- Made the heap size of the forked test process configurable via a
property. This was necessary because running the whole build requires about 4gb
of memory for the test process, but 4gb is too large for the Travis container
and the smoke tests only require 3.5gb
- Cached maven dependencies for the Travis build so that it runs faster
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ilooner/drill DRILL-5989
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/1053.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1053
----
commit c5274ed3890c9bf116a79043e7610f9d9a968bd6
Author: Timothy Farkas <[email protected]>
Date: 2017-11-22T23:02:07Z
- Categories some tests to speed up smoke tests
- Made travis run tests
commit f565e64f7c8f7cf95d6747a2d651979bf4d868bc
Author: Timothy Farkas <[email protected]>
Date: 2017-11-22T23:06:47Z
- Cache maven artifacts
commit a65e1c1124408b53b376b6a7d4419b38b91a80b2
Author: Timothy Farkas <[email protected]>
Date: 2017-11-23T00:43:44Z
- Fixed missing class exception
commit f608992b66c2a9347582e5724d11f8bf0e434dda
Author: Timothy Farkas <[email protected]>
Date: 2017-11-27T18:16:46Z
- Added drill common test classes
commit 6b05a1d2d455e17f78f568f49035e4227ec46686
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T00:20:20Z
- Fixed bug where excluded tests are run anyways
- Reduced runtime of smoke tests by categorizing more long running,
unlikely, and security tests.
- Deleted commented out code
- Fixed class not found exception in the jdbc-all tests
commit 53c70ca598f9821faba2773ff1a743c901a76eb6
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T01:30:55Z
- Fixed classpath for tests
commit f87882693b9f9cc571e7335542d831fbe3d79d00
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T20:02:05Z
- Categorized kafka tests
- Increased memory size
commit c594b1164834f5d24e324a227a72edee953e585b
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T20:02:52Z
- Added missing file
commit 6e0cb95a492d4802a5c85818455274b8525459e5
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T20:11:58Z
- Fixed memory settings again
commit 474cba2915a883ac559c8071a3016cd854e42e63
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T21:15:26Z
- Increased memory for travis
commit b59a274511c9d125ee242285c3dee7035577fd4f
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T22:18:41Z
- Ignore broken tests
commit fd7ae8f378e9a87cad1f46cfc269709fb1431426
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T22:37:07Z
- Fixed yml syntax error
commit ee57dd7c08f3ca7169885bc1993003e05e385610
Author: Timothy Farkas <[email protected]>
Date: 2017-11-28T23:20:47Z
- More memory config fixes
commit 365f2b58b36748753a42fb6d85792e1f5d5a9552
Author: Timothy Farkas <[email protected]>
Date: 2017-11-29T00:22:53Z
- Minor code cleanup
----
---
