[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