Skip to content

Use provider equality in the exec transition cache - #31030

Open
fmeum wants to merge 1 commit into
bazelbuild:masterfrom
fmeum:exec-transition-cache-equality
Open

Use provider equality in the exec transition cache#31030
fmeum wants to merge 1 commit into
bazelbuild:masterfrom
fmeum:exec-transition-cache-equality

Conversation

@fmeum

@fmeum fmeum commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Description

Don't use hashCode for the exec transition cache as it is prone to conflicts.

Motivation

Also prepares for planned future changes to hashCode that may not include all fields.

Build API Changes

No

Checklist

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

Release Notes

RELNOTES: None

@fmeum
fmeum force-pushed the exec-transition-cache-equality branch 3 times, most recently from 4fbf4c7 to b2ee4cf Compare September 6, 2026 15:03
@fmeum
fmeum marked this pull request as ready for review September 6, 2026 15:03
@fmeum
fmeum requested a review from a team as a code owner September 6, 2026 15:03
@fmeum
fmeum requested review from aranguyen and dabanki and removed request for a team and aranguyen September 6, 2026 15:03
@github-actions github-actions Bot added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-review PR is awaiting review from an assigned reviewer labels Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer team-Configurability platforms, toolchains, cquery, select(), config transitions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants