[ https://issues.apache.org/jira/browse/ARROW-2083?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16356639#comment-16356639 ]
ASF GitHub Bot commented on ARROW-2083: --------------------------------------- pitrou commented on a change in pull request #1568: ARROW-2083: [CI] Detect changed components on Travis-CI URL: https://github.com/apache/arrow/pull/1568#discussion_r166860298 ########## File path: .travis.yml ########## @@ -61,94 +63,110 @@ matrix: - export CXX="clang++-4.0" - $TRAVIS_BUILD_DIR/ci/travis_install_clang_tools.sh - $TRAVIS_BUILD_DIR/ci/travis_lint.sh - - $TRAVIS_BUILD_DIR/ci/travis_before_script_cpp.sh + - if [ $ARROW_CI_CPP_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_before_script_cpp.sh; fi script: - - $TRAVIS_BUILD_DIR/ci/travis_script_cpp.sh - - $TRAVIS_BUILD_DIR/ci/travis_build_parquet_cpp.sh - - $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 2.7 - - $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 3.6 + - if [ $ARROW_CI_CPP_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_cpp.sh; fi + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_build_parquet_cpp.sh; fi + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 2.7; fi + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 3.6; fi + # [OS X] C++ & Python w/ XCode 6.4 - compiler: clang language: cpp osx_image: xcode6.4 os: osx cache: addons: before_script: + - eval `python $TRAVIS_BUILD_DIR/ci/travis_detect_changes.py` - export ARROW_TRAVIS_USE_TOOLCHAIN=1 - export ARROW_TRAVIS_PLASMA=1 - export ARROW_TRAVIS_ORC=1 - export ARROW_BUILD_WARNING_LEVEL=CHECKIN - - travis_wait 50 $TRAVIS_BUILD_DIR/ci/travis_before_script_cpp.sh + - if [ $ARROW_CI_CPP_AFFECTED == "1" ]; then travis_wait 50 $TRAVIS_BUILD_DIR/ci/travis_before_script_cpp.sh; fi script: - - $TRAVIS_BUILD_DIR/ci/travis_script_cpp.sh - - $TRAVIS_BUILD_DIR/ci/travis_build_parquet_cpp.sh - - $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 2.7 - - $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 3.6 + - if [ $ARROW_CI_CPP_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_cpp.sh; fi + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_build_parquet_cpp.sh; fi + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 2.7; fi + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_python.sh 3.6; fi + # [manylinux1] Python - language: cpp before_script: - - docker pull quay.io/xhochy/arrow_manylinux1_x86_64_base:latest + - eval `python $TRAVIS_BUILD_DIR/ci/travis_detect_changes.py` + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then docker pull quay.io/xhochy/arrow_manylinux1_x86_64_base:latest; fi script: - - $TRAVIS_BUILD_DIR/ci/travis_script_manylinux.sh + - if [ $ARROW_CI_PYTHON_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_manylinux.sh; fi + # Java w/ OpenJDK 7 - language: java os: linux jdk: openjdk7 + before_script: + - eval `python $TRAVIS_BUILD_DIR/ci/travis_detect_changes.py` script: - - $TRAVIS_BUILD_DIR/ci/travis_script_java.sh + - if [ $ARROW_CI_JAVA_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_java.sh; fi + - if [ $ARROW_CI_SITE_AFFECTED == "1" ]; then $TRAVIS_BUILD_DIR/ci/travis_script_site.sh; fi Review comment: Oh, I see. I thought `mvn site` rebult the whole Web site. Apparently I was mistaken. ---------------------------------------------------------------- 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: us...@infra.apache.org > Support skipping builds > ----------------------- > > Key: ARROW-2083 > URL: https://issues.apache.org/jira/browse/ARROW-2083 > Project: Apache Arrow > Issue Type: Improvement > Reporter: Uwe L. Korn > Assignee: Antoine Pitrou > Priority: Major > Labels: pull-request-available > > While appveyor supports a [skip appveyor] you cannot skip only travis. What > is the feeling about adding e.g. > [https://github.com/travis-ci/travis-ci/issues/5032#issuecomment-273626567] > to our build. We could also do some simple kind of change detection that we > don't build the C++/Python parts and only Java and the integration tests if > there was a change in the PR that only affects Java. > I think it might be worthwhile to spend a bit on that to get a bit of load of > the CI infrastructure. -- This message was sent by Atlassian JIRA (v7.6.3#76005)