Skip to content

[9.1.0] Reduce Merkle tree footprint by lazily converting to Paths (https:/…#28831

Merged
iancha1992 merged 1 commit intobazelbuild:release-9.1.0from
iancha1992:cp28812
Mar 3, 2026
Merged

[9.1.0] Reduce Merkle tree footprint by lazily converting to Paths (https:/…#28831
iancha1992 merged 1 commit intobazelbuild:release-9.1.0from
iancha1992:cp28812

Conversation

@iancha1992
Copy link
Member

…/github.com//pull/28737)

Paths of ActionInputs are typically not retained, which means that keeping them in memory as part of a Merkle tree is wasteful. Instead, retain the inputs and lazily convert to Path when uploading using the ArtifactPathResolver anyway retained by SpawnExecutionContext.

Work towards #20478
Work towards #28734

No

  • I have added tests for the new use cases (if any).
  • I have updated the documentation (if applicable).

RELNOTES: None

Closes #28737.

PiperOrigin-RevId: 876039610
Change-Id: Id4af6af2daf7ef72e8a8e3da35f00e2dccd81bd0

Description

Motivation

Build API Changes

No

Checklist

  • I have added tests for the new use cases (if any).
  • I have updated the documentation (if applicable).

Release Notes

RELNOTES: None

Commit 21ac6c1

…ild#28737)

`Path`s of `ActionInput`s are typically not retained, which means that keeping them in memory as part of a Merkle tree is wasteful. Instead, retain the inputs and lazily convert to `Path` when uploading using the `ArtifactPathResolver` anyway retained by `SpawnExecutionContext`.

Work towards bazelbuild#20478
Work towards bazelbuild#28734

No

- [ ] I have added tests for the new use cases (if any).
- [ ] I have updated the documentation (if applicable).

RELNOTES: None

Closes bazelbuild#28737.

PiperOrigin-RevId: 876039610
Change-Id: Id4af6af2daf7ef72e8a8e3da35f00e2dccd81bd0
@iancha1992 iancha1992 requested a review from tjgq February 27, 2026 19:38
@iancha1992 iancha1992 requested a review from a team as a code owner February 27, 2026 19:38
@iancha1992 iancha1992 added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Feb 27, 2026
@iancha1992 iancha1992 enabled auto-merge February 27, 2026 19:38
@iancha1992 iancha1992 added this pull request to the merge queue Mar 3, 2026
Merged via the queue into bazelbuild:release-9.1.0 with commit 59d11b3 Mar 3, 2026
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants