[GitHub] [incubator-druid] ccaominh opened a new pull request #8240: WIP Speedup Travis CI jobs

[GitBox <gi...@apache.org>]

ccaominh opened a new pull request #8240: WIP Speedup Travis CI jobs
URL: https://github.com/apache/incubator-druid/pull/8240
 
 
   _**NOTE: This PR is a work-in-progress to see effects of the Travis CI changes.**_
   
   ### Description
   
   Reorganize Travis CI jobs into smaller (and more) jobs. Add various
   maven options to skip unnecessary work and refactored Travis CI job
   definitions to follow DRY.
   
   #### Detailed changes:
   
   .travis.yml
   - Refactor build logic to get rid of copy-and-paste logic
   - Skip static checks and enable parallelism for maven install
   - Split static analysis into different jobs to ease triage
   - Use "name" attribute instead of NAME environment variable
   - Split "indexing" and "web console" out of "other modules test"
   - Split 2 integration test jobs into multiple smaller jobs
   
   build.sh
   - Enable parallelism
   - Disable more static checks
   
   travis_script_integration.sh
   travis_script_integration_part2.sh
   integration-tests/README.md
   - Use TestNG groups instead of shell scripts and move definition of jobs
     into Travis CI yaml
   
   integration-tests/pom.xml
   - Show elapsed time of individual tests to aid in rebalancing Travis CI
     integration test jobs run time
   
   TestNGGroup.java
   - Use TestNG groups to make it easy to have multiple Travis CI
     integration test jobs. TestNG groups also make it easier to have an
     "other" integration test group and makes it less likely a test will
     accidentally not be included in a CI job.
   
   ITHadoopIndexTest.java
   AbstractITBatchIndexTest.java
   AbstractKafkaIndexerTest.java
   ITAppenderatorDriverRealtimeIndexTaskTest.java
   ITCompactionTaskTest.java
   ITIndexerTest.java
   ITKafkaIndexingServiceTest.java
   ITKafkaIndexingServiceTransactionalTest.java
   ITNestedQueryPushDownTest.java
   ITParallelIndexTest.java
   ITRealtimeIndexTaskTest.java
   ITSystemTableBatchIndexTaskTest.java
   ITUnionQueryTest.java
   ITSystemTableQueryTest.java
   ITTwitterQueryTest.java
   ITWikipediaQueryTest.java
   ITBasicAuthConfigurationTest.java
   ITCoordinatorOverlordProxyAuthTest.java
   ITTLSTest.java
   - Add TestNG group
   - Fix various IntelliJ inspection warnings
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   - [x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
   - [x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org