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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org