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

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed

yes, 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

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.