Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARROW-6751: [CI] Fix ccache setup on Travis-CI #5554

Closed
wants to merge 3 commits into from

Conversation

pitrou
Copy link
Member

@pitrou pitrou commented Oct 1, 2019

No description provided.

@pitrou
Copy link
Member Author

pitrou commented Oct 1, 2019

Results on my Travis-CI account:

For some reason, it seems caching doesn't work on OS X builders?

@github-actions
Copy link

github-actions bot commented Oct 1, 2019

@pitrou
Copy link
Member Author

pitrou commented Oct 1, 2019

Well, of course, the reason "caching doesn't work on OS X builders" is that we disable it.

@pitrou pitrou force-pushed the ARROW-6751-travis-ccache branch 3 times, most recently from 37862aa to 736af75 Compare October 1, 2019 19:54
@pitrou
Copy link
Member Author

pitrou commented Oct 1, 2019

Full Travis-CI build: https://travis-ci.org/pitrou/arrow/builds/592231651

For comparison, without cache: https://travis-ci.org/apache/arrow/builds/592020803

Will merge once green.

@pitrou pitrou closed this in 871aedb Oct 1, 2019
@pitrou pitrou deleted the ARROW-6751-travis-ccache branch October 1, 2019 22:01
@pitrou
Copy link
Member Author

pitrou commented Oct 1, 2019

As the comparison above shows, 1.5 hour of CPU time was saved on a full build with warm cache.

kszucs pushed a commit that referenced this pull request Oct 5, 2019
Closes #5554 from pitrou/ARROW-6751-travis-ccache and squashes the following commits:

cbc10f7 <Antoine Pitrou> Again
08857c5 <Antoine Pitrou> Again
4ea772d <Antoine Pitrou> ARROW-6751:  Fix ccache setup on Travis-CI

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant