Skip to content

Convert access graph resources to new in memory cache collection#54481

Merged
rosstimothy merged 1 commit intomasterfrom
tross/cache_tag_resources
May 5, 2025
Merged

Convert access graph resources to new in memory cache collection#54481
rosstimothy merged 1 commit intomasterfrom
tross/cache_tag_resources

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Moves crown jewels and access graph settings to the new cache collection scheme that was introduced in #52210. No additional functionality changes have been made here. This should be a purely mechanical translation to the new internal caching machinery.

@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label May 2, 2025
@rosstimothy rosstimothy force-pushed the tross/cache_tag_resources branch 3 times, most recently from 79f7361 to fc3a8b4 Compare May 2, 2025 17:42
@rosstimothy rosstimothy marked this pull request as ready for review May 2, 2025 18:23
@github-actions github-actions Bot requested review from camscale and espadolini May 2, 2025 18:23
@rosstimothy rosstimothy requested a review from fspmarshall May 2, 2025 18:45
Moves crown jewels and access graph settings to the new cache
collection scheme that was introduced in #52210. No additional
functionality changes have been made here. This should be a
purely mechanical translation to the new internal caching machinery.
@rosstimothy rosstimothy force-pushed the tross/cache_tag_resources branch from fc3a8b4 to 5406edf Compare May 5, 2025 20:19
@rosstimothy rosstimothy enabled auto-merge May 5, 2025 20:19
@rosstimothy rosstimothy added this pull request to the merge queue May 5, 2025
Merged via the queue into master with commit d659853 May 5, 2025
40 checks passed
@rosstimothy rosstimothy deleted the tross/cache_tag_resources branch May 5, 2025 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants