Skip to content

MINOR: [C++][CI] Bump ccache max size - #51259

Open
pitrou wants to merge 1 commit into
apache:mainfrom
pitrou:minor-ccache-size
Open

MINOR: [C++][CI] Bump ccache max size#51259
pitrou wants to merge 1 commit into
apache:mainfrom
pitrou:minor-ccache-size

Conversation

@pitrou

@pitrou pitrou commented Sep 9, 2026

Copy link
Copy Markdown
Member

Rationale for this change

When trying to analyze the compilation caching behavior on #51255, I realized that our maximum ccache size of 1GB had become too small for some builds. The result is that a single build cannot be cached in its entirety, making identical rebuilds costlier than they should be.

What changes are included in this PR?

Bump ccache max size to 1.5 GB.

Are these changes tested?

Yes, by existing builds.

Are there any user-facing changes?

No.

@github-actions github-actions Bot added the awaiting review Awaiting review label Sep 9, 2026
When trying to analyze the compilation caching behavior on apache#51255, I realized that our maximum ccache size of 1GB had become too small for some builds. The result is that a single build cannot be cached in its entirety, making identical rebuilds costlier than they should be.
@pitrou pitrou added the CI: Extra: C++ Run extra C++ CI label Sep 9, 2026
@pitrou
pitrou marked this pull request as ready for review September 9, 2026 13:53
@pitrou

pitrou commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

cc @kou @raulcd

@raulcd

raulcd commented Sep 9, 2026

Copy link
Copy Markdown
Member

Is there any specific log to look for to see this happening in the future? I am unsure I know how to find that this cache size has become too small.

@pitrou

pitrou commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

What I did is run a container locally, issued ccache -Cz to clear the cache and its statistics, then ran the compilation and looked at the ccache stats at the end.

I'm afraid there's no easy way to automate such a hint on CI builds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review Awaiting review CI: Extra: C++ Run extra C++ CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants