Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix #4545: clear the cache after site is updated of deleted. #4546

Closed
wants to merge 1 commit into from

Conversation

zyhfish
Copy link
Contributor

@zyhfish zyhfish commented Aug 22, 2024

No description provided.

@sbwalker
Copy link
Member

@zyhfish cache invalidation for site updates is already handled in CacheInvalidationEventSubscriber - can you please describe the scenario where it is not working so I can reproduce the issue.

@sbwalker
Copy link
Member

sbwalker commented Aug 22, 2024

@zyhfish the cache invalidation on a delete operation should not be necessary as it will automatically timeout in 20 minutes if it is not referenced - however I suppose in order to free up the memory immediately it could be added to CacheInvalidationEventSubscriber

@sbwalker sbwalker closed this in f6858c2 Aug 22, 2024
sbwalker added a commit that referenced this pull request Aug 22, 2024
fix #4546 - handle cache invalidation for site deletion
@sbwalker
Copy link
Member

I merged #4548 to handle the site deletion scenario

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants