Skip to content

Conversation

@coryan
Copy link
Contributor

@coryan coryan commented Dec 19, 2017

This is not a huge speedup, but noticeable. Right now we are
building all the grpc, abseil, and cctz targets. With this change
only the targets that we need for our dependencies are built.

This is not a huge speedup, but noticeable.  Right now we are
building all the grpc, abseil, and cctz targets.  With this change
only the targets that we need for our dependencies are built.
I think we get around 30% faster builds with this.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 19, 2017
Copy link
Contributor

@mbrukman mbrukman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

What about caching outputs on a per-build-job basis?

@coryan coryan merged commit 3bbf36b into googleapis:master Dec 19, 2017
@coryan coryan deleted the speedup-travis-builds branch December 19, 2017 20:42
@coryan
Copy link
Contributor Author

coryan commented Dec 19, 2017

Created #118 for the caching builds. I think we need to close #49 first.

coryan added a commit that referenced this pull request Jun 1, 2018
Part of the fixes for #19, and also helps with #116.
@alevenberg alevenberg mentioned this pull request Dec 28, 2023
bajajneha27 added a commit to bajajneha27/google-cloud-cpp that referenced this pull request Apr 29, 2025
bajajneha27 added a commit to bajajneha27/google-cloud-cpp that referenced this pull request Apr 30, 2025
ddelgrosso1 added a commit that referenced this pull request Apr 30, 2025
…RTED as retriable error (#15114)

* chore(ACv2): Add integration tests for fastbyte operations (#115)

* feat: Make ABORTED as retriable error (#116)

* test failure fix and address review comments

* Improve some internal documentation

* checkers-pr fix

---------

Co-authored-by: Denis DelGrosso <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants