[
https://issues.apache.org/jira/browse/ARROW-2083?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16356287#comment-16356287
]
ASF GitHub Bot commented on ARROW-2083:
---------------------------------------
wesm 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_r166802521
##########
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:
I believe this should be run whenever Java is changed -- this tests that the
Javadoc generation is not broken
----------------------------------------------------------------
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:
[email protected]
> 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)
