You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Antoine Pitrou (JIRA)" <ji...@apache.org> on 2018/02/21 10:13:04 UTC

[jira] [Commented] (ARROW-2192) Commits to master should run all builds in CI matrix

    [ https://issues.apache.org/jira/browse/ARROW-2192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16371177#comment-16371177 ] 

Antoine Pitrou commented on ARROW-2192:
---------------------------------------

I see... So we should be able to decide this based on the environment variables exported by Travis-CI :

https://docs.travis-ci.com/user/environment-variables/#Convenience-Variables

> Commits to master should run all builds in CI matrix
> ----------------------------------------------------
>
>                 Key: ARROW-2192
>                 URL: https://issues.apache.org/jira/browse/ARROW-2192
>             Project: Apache Arrow
>          Issue Type: Bug
>            Reporter: Wes McKinney
>            Priority: Major
>             Fix For: 0.9.0
>
>
> After ARROW-2083, we are only running builds related to changed components with each patch in Travis CI and Appveyor. 
> The problem with this is that when we merge patches to master, our Travis CI configuration (implemented by ASF infra to help alleviate clogged up build queues) is set up to cancel in-progress builds whenever a new commit is merged.
> So basically we could have in our timeline:
> * Patch merged affecting C++, Python
> * Patch merged affecting Java
> * Patch merged affecting JS
> So when the Java patch is merged, any in-progress C++/Python builds will be cancelled. And if the JS patch comes in, the Java builds would be immediately cancelled.
> In light of this I believe on master branch we should always run all of the builds unconditionally



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)