Skip to content

Improve --execution_log_compact_file= with output service - #30498

Closed
keith wants to merge 2 commits into
bazelbuild:masterfrom
keith:ks/improve-execution_log_compact_file-with-output-service
Closed

Improve --execution_log_compact_file= with output service#30498
keith wants to merge 2 commits into
bazelbuild:masterfrom
keith:ks/improve-execution_log_compact_file-with-output-service

Conversation

@keith

@keith keith commented Jul 28, 2026

Copy link
Copy Markdown
Member

Previously since the logging code wasn't aware of output services it
would inspect every file for the log info, resulting in downloads of
every file. Now we use the output service for this. In my test this took
a build from ~10 minutes to ~4 minutes with bb-clientd (based on network quality)

Previously since the logging code wasn't aware of output services it
would inspect every file for the log info, resulting in downloads of
every file. Now we use the output service for this. In my test this took
a build from ~10 minutes to ~4 minutes with bb-clientd (based on network quality)
@github-actions github-actions Bot added the awaiting-review PR is awaiting review from an assigned reviewer label Jul 28, 2026
@fmeum
fmeum requested a review from coeuvre July 28, 2026 21:43
@fmeum

fmeum commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

@bazel-io fork 9.3.0

@github-actions github-actions Bot added the community-reviewed Reviewed by a trusted community contributor label Jul 28, 2026
@iancha1992 iancha1992 added the team-Remote-Exec Issues and PRs for the Execution (Remote) team label Jul 28, 2026
@coeuvre coeuvre added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Jul 29, 2026
@github-actions github-actions Bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-reviewed Reviewed by a trusted community contributor team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants