Skip to content

feat: default builder cache sharing on#5529

Merged
klkvr merged 9 commits into
mainfrom
centaur/1781020071-1677-3121
Jun 11, 2026
Merged

feat: default builder cache sharing on#5529
klkvr merged 9 commits into
mainfrom
centaur/1781020071-1677-3121

Conversation

@decofe

@decofe decofe commented Jun 9, 2026

Copy link
Copy Markdown
Member

Summary

  • keep --builder.enable-prewarming accepted as a no-op legacy flag, defaulting to true for compatibility
  • keep execution cache sharing with the payload builder enabled by default
  • add --engine.disable-execution-cache-sharing-with-builder for explicit opt-out
  • add CLI/defaults coverage for the cache-sharing opt-out and legacy prewarming flag behavior

Testing

  • cargo fmt
  • cargo test -p tempo --bin tempo consensus_block_budget_defaults_are_stable

Prompted by: @shekhirin

@decofe decofe force-pushed the centaur/1781020071-1677-3121 branch from de3ce51 to f8bf637 Compare June 9, 2026 16:14
@shekhirin shekhirin changed the title Enable builder prewarming by default feat: enable builder prewarming and cache by default Jun 10, 2026
@decofe decofe changed the title feat: enable builder prewarming and cache by default feat: default builder cache sharing on Jun 11, 2026
@shekhirin shekhirin force-pushed the centaur/1781020071-1677-3121 branch from 6a19bf6 to 7b0a9b0 Compare June 11, 2026 16:30
klkvr
klkvr previously approved these changes Jun 11, 2026
…77-3121

# Conflicts:
#	bin/tempo/src/main.rs
klkvr
klkvr previously approved these changes Jun 11, 2026
@tempo-voight-kampff

Copy link
Copy Markdown

Hi @klkvr — your review approval was detected by Voight-Kampff, but you rejected the approval request. Your review did not count toward this PR.

The prompt was canceled.

This request was for head 9a27195c9d25.

Re-submit the review and complete the new Voight-Kampff prompt if you want this review to count.

@klkvr klkvr added this pull request to the merge queue Jun 11, 2026
Merged via the queue into main with commit 789778f Jun 11, 2026
35 checks passed
@klkvr klkvr deleted the centaur/1781020071-1677-3121 branch June 11, 2026 18:49
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.

3 participants