[wp-trac] [WordPress Trac] #37189: In wp_term_query on cache ids

[WordPress Trac]

#37189: In wp_term_query on cache ids
----------------------------------------+-----------------------------
 Reporter:  spacedmonkey                |       Owner:
     Type:  enhancement                 |      Status:  new
 Priority:  normal                      |   Milestone:  Future Release
Component:  Taxonomy                    |     Version:  4.6
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:  performance
----------------------------------------+-----------------------------
Changes (by spacedmonkey):

 * keywords:  needs-patch => has-patch needs-unit-tests


Comment:

 @boonebgorges thanks for the feedback. My latest patch is a rewrite so I
 am not sure that you feedback is valid. The new patch does the following.

 - There are now only two query types, everything and count. This makes the
 code much more simple.
 - All caches now are stored as a list of ids, no matter the format.
 - The cache key is generated differently. As all query are cached as a
 list of ids, the same cache can be used for all formats of return (apart
 from count). This means for example, two queries one with fields=>'ids'
 and one with fields=>'all' they will hit the same cache. This could be a
 massive performance win for many :D

 Looping @flixos90 into this ticket, as he did the work on wp_term_query.

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