Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi! Have you thought about leaving the cache_clear_all() calls from variable_del() and _set() until, say, an implementation of hook_exit()? Maybe this would improve performance under some cases.
Comments
Comment #1
Anonymous (not verified) CreditAttribution: Anonymous commentedyes, i've thought of moving them, but not for performance reasons. we can't group the cache clear calls because there's not cache->deleteMultiple($cids);
however, there's still a race with the current implementation, and moving the cache_clear_all() calls later in the request will make it less likely to occur.
committed here:
http://drupalcode.org/project/variables_that_suck_less.git/commit/97ff94a