You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2018/06/05 06:46:24 UTC

[GitHub] sijie opened a new pull request #1481: Improve precommit build time

sijie opened a new pull request #1481: Improve precommit build time
URL: https://github.com/apache/bookkeeper/pull/1481
 
 
   
   Descriptions of the changes in this PR:
   
   *Motivation*
   
   The build/test time of `bookkeeper-server` is almost close to or more than 1 hr.
   
   *Solution*
   
   Split the precommit jobs into multiple jobs:
   
   - *PR Validation* : run `apache-rat:check`, `checkstyle:check`, to ensure codestyle and license headers
   - *Build (Java x)* : run `package` `spotbugs:check`, to ensure PR can be compiled and tested on different java environments. No spotbugs errors. `bookkeeper-server` tests are skipped.
   - *Integration Tests* : run integration tests with docker environment
   - *bookkeeper-server Tests* : tests `bookkeeper-server` on certain packages:
     - bookie: `org.apache.bookkeeper.bookie`
     - client: `org.apache.bookkeeper.client`
     - replication: `org.apache.bookkeeper.replication`
     - tls: `org.apache.bookkeeper.tls`
     - all others: all other tests in `bookkeeper-server` module
   
   Each job can be triggered and skipped individually.
   
   - *PR Validation* : "(re)run pr validation" | "" (unskippable)
   - *Build (Java x)* : "(re)build java(8|9)" | "skip build java(8|9)"
     - "(re)build" to build on both java8 and java9
     - "skip build" to skip building jobs on both java8 and java9
   - *Integration Tests* : "(re)run integration tests" | "skip integration tests"
   - *bookkeeper-server tests* : "(re)run (bookie|client|replication|tls|remaining) tests" | "skip (bookie|client|replication|tls|remaining) tests"
     - "(re)run tests" to run all bookkeeper-server test jobs
     - "skip tests" to skip all bookkeeper-server test jobs
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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