Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

Try Travis caching of jars #591

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Try Travis caching of jars #591

wants to merge 2 commits into from

Conversation

johnynek
Copy link
Collaborator

should speed up builds, potentially.

@sritchie
Copy link
Collaborator

You can check if it worked once the tests pass by running travis cache at the command line.

@sritchie
Copy link
Collaborator

Well, I guess the tests have been erroring for quite some time. Maybe this isn't the best project to test this opt on.

@johnynek
Copy link
Collaborator Author

yeah, right now the tests are pretty hosed. fracking storm.

@sritchie
Copy link
Collaborator

realtime is the future. Any negativity or inference that there might be a problem with Storm is a sign of impending codgerhood.

@sritchie
Copy link
Collaborator

@johnynek merged develop since tests now pass. Pushed again.

@johnynek
Copy link
Collaborator Author

@sritchie thanks old friend.

@sritchie
Copy link
Collaborator

I'm seeing

Setting up build cache
$ export CASHER_DIR=$HOME/.casher
0.03s$ Installing caching utilities
0.48sattempting to download cache archive
found cache

Unclear how much time's getting saved on the build.

@johnynek
Copy link
Collaborator Author

total of this run was 58 min, whereas this was 57 min: #601

So far, no evidence of a win.

@sritchie
Copy link
Collaborator

Or if your metric is "distance away from a 1 hour run time", I'm seeing 50% improvement :)

@johnynek
Copy link
Collaborator Author

:)

snoble pushed a commit to snoble/summingbird that referenced this pull request Sep 8, 2017
optimize Generated{Abstract,Product}Algebra with benchmarking
@CLAassistant
Copy link

CLAassistant commented Jul 18, 2019

CLA assistant check
All committers have signed the CLA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants