[wp-trac] [WordPress Trac] #53804: Bug: `wp_cache_themes_persistently` only used for setting, not getting

[WordPress Trac]

#53804: Bug: `wp_cache_themes_persistently` only used for setting, not getting
--------------------------+------------------------------
 Reporter:  mattwiebe     |       Owner:  (none)
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Themes        |     Version:
 Severity:  normal        |  Resolution:  invalid
 Keywords:  needs-patch   |     Focuses:  administration
--------------------------+------------------------------
Changes (by mattwiebe):

 * status:  new => closed
 * resolution:   => invalid


Comment:

 Thanks for the speedy feedback @SergeyBiryukov ! I was close to EOD and
 just wanted to get this up after a frustrating day of debugging the
 underlying issue. I should have waited a little longer and filed this with
 more calm investigation.

 I've discovered that this is actually a bug in our memcached backend,
 which does not respect the call to `wp_cache_add_non_persistent_groups(
 'themes' )` as it should. (It just stubs out the function and `return`s
 silently). So all the `cache_get` and `cache_add` calls still wind up in
 the persisted group rather than simply being persisted in-memory as
 expected from the filter value.

 Thanks again!

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53804#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform