Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: added disk cache versioning #3440

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

davidejensen
Copy link
Member

Pull Request Description

What does this PR change?

This PR introduces Disk Cache versioning to allow full wipe of old cache when introducing major changes like partial loading and partial disk cache

@davidejensen davidejensen self-assigned this Feb 26, 2025
@davidejensen davidejensen requested review from a team as code owners February 26, 2025 09:55
Copy link
Contributor

@NickKhalow NickKhalow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with comments

@@ -6,6 +6,9 @@ namespace ECS.StreamableLoading.Cache.Disk
{
public readonly struct CacheDirectory
{
//Bump this version if there is the need to wipe out the old disk cache when introducing major changes to DiskCaching
private const string CACHE_VERSION = "V1";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can be int and "V" mode to DISK_CACHE_FOLDER name itself

@davidejensen davidejensen merged commit 75b1e78 into release/release-25-02-2025 Feb 26, 2025
7 checks passed
@davidejensen davidejensen deleted the fix/cache-versioning branch February 26, 2025 11:22
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.

2 participants