Avoid using variable_set() for storing cache flush times. #926062: Avoid variable_set() on bin and wildcard flushes

Comments

crea’s picture

Title: Optimise minimum cache lifetime mode » Avoid using variable_set() for storing cache flush times.

removed

catch’s picture

fwiw I went a different way in memcache, adding memcache-specific versions of variable set so that it's write through.

There were two main reasons for this:

- storing it in memcache risks losing it if there's an eviction or partial restart of a memcache cluster. That's not an issue for mongo though.
- having an extra request to memcache to check expiry had a noticeable performance impact on raw page caching performance. This makes sense since you're talking about only a handful (or in D7 potentially 1) requests - so adding one is not small in that context.

catch’s picture

Issue summary: View changes

Removed separate issue info

fgm’s picture

Version: 7.x-1.x-dev » 8.x-2.x-dev
Issue summary: View changes

This was a good idea, but the D7 version of the module won't evolve now that Drupal core is already at 8.5.

Might make sense after rethinking what it could mean on D8.

fgm’s picture