[wp-trac] [WordPress Trac] #31245: Replace alloptions with a key cache
WordPress Trac
noreply at wordpress.org
Sat May 6 14:21:33 UTC 2017
#31245: Replace alloptions with a key cache
-------------------------------------+-----------------------------
Reporter: rmccue | Owner: rmccue
Type: enhancement | Status: assigned
Priority: normal | Milestone: Future Release
Component: Options, Meta APIs | Version:
Severity: normal | Resolution:
Keywords: has-patch needs-testing | Focuses: performance
-------------------------------------+-----------------------------
Comment (by spacedmonkey):
In 31245.3.diff I try something a little different. I really like the
changed in 31245.2.diff however, I don't think that is fixes the race
condition, where autoloaded options revert. In my patch just deleted the
alloptions cache value after add/update/delete options. It treats
alloptions cache as poisoned on update and deletes it. I believe this is
the best and most simple solution to this error. It is also how the meta
api works. https://github.com/WordPress/WordPress/blob/master/wp-
includes/meta.php#L107
As for WP_Option and WP_Option_Query, this is great work and should be
broken into another ticket.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/31245#comment:50>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list