[wp-trac] [WordPress Trac] #31245: Replace alloptions with a key cache
WordPress Trac
noreply at wordpress.org
Fri May 12 19:52:03 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):
My recommendation is the following.
We use my patch 31245.3.diff , as it fixes the race condition documented
at above. It is a tidy solution as it doesn't require the introduction of
get_cache_multi and possible breaks. I would like @rmccue & @joehoyle to
take a look at my patch, as I know it is issue they have looked and fixed
outside of core.
As for WP_Option_Query and WP_Option, I like standardising interfaces in
core. WP_Option_Query feels like a good fit for the api, as there is value
in search for options by name. WP_Option_Query has some nice filters and
should replace the internals of wp_load_alloptions. For that reason,
WP_Option_Query work should be moved to #33958 .
--
Ticket URL: <https://core.trac.wordpress.org/ticket/31245#comment:53>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list