Skip to content

Enable configuration cache #195

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

Merged
merged 5 commits into from
Jul 2, 2025
Merged

Enable configuration cache #195

merged 5 commits into from
Jul 2, 2025

Conversation

simolus3
Copy link
Contributor

Using the configuration cache can speed up builds by caching results of the configuration phase, which means that Gradle doesn't have to evaluate buildscripts multiple times.

An even bigger impact is that enabling this allows tasks in the same project to run in parallel, which is particularly helpful when working on the SDK: The steps to compile SQLite for :static-sqlite-driver are sped up a lot by running them in parallel (by default, we'd compile SQLite one ABI at a time).

I've heard that the configuration cache doesn't work too well with some publishing plugins, so I've disabled it for the publishing CI. Apart from that, this should hopefully speed up our builds. We could cache the results of the configuration cche between CI runs too by setting a key to encrypt them with (it's not cached by default because the data may be sensitive). But I think the benefits of multi-threading alone could justify this and we don't necessarily need to worry about that.

Base automatically changed from build-cache to main May 22, 2025 10:10
@simolus3 simolus3 force-pushed the configuration-cache branch from 938414e to ab72541 Compare July 1, 2025 17:00
@simolus3 simolus3 changed the title WIP: Try enabling configuration cache Enable configuration cache Jul 1, 2025
@simolus3 simolus3 requested a review from stevensJourney July 1, 2025 21:11
@simolus3 simolus3 marked this pull request as ready for review July 1, 2025 21:11
@simolus3 simolus3 force-pushed the configuration-cache branch from 5298f7d to 658b5a3 Compare July 2, 2025 14:53
Copy link
Contributor

@stevensJourney stevensJourney left a comment

Choose a reason for hiding this comment

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

Looks good to me :)

@simolus3 simolus3 merged commit e5300be into main Jul 2, 2025
3 checks passed
@simolus3 simolus3 deleted the configuration-cache branch July 2, 2025 15:04
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.

2 participants