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.

Sign up for GitHub

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

Jump to bottom

Optimize checkForCacheUpdate to avoid deleting the same cache key multiple times #512

Open
wants to merge 1 commit into
base: hotfix
Choose a base branch
from
Open

Optimize checkForCacheUpdate to avoid deleting the same cache key multiple times #512

wants to merge 1 commit into from

Commits on Jul 18, 2024

  1. Optimize checkForCacheUpdate to avoid deleting the same cache key mul… 

    …tiple times

In some installations, the same cache key may be multiple times in the `cache_rebuild` table. Therefore, the same `DELETE` statement will be executed multiple times, even though it would only be necessary to be executed once, to delete that key from the table. (E.g. `DELETE FROM cache_rebuild WHERE cache_key='app-metadata-theme-images'`). This change optimizes the loop that iterates over the keys to be deleted, and keeps track of those keys already deleted, to skip any additional superfluous deletions.
    @LoLei
    LoLei committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    756ea13 View commit details
    Browse the repository at this point in the history