[wp-trac] [WordPress Trac] #22176: Cache the results of the posts_request_ids query

WordPress Trac noreply at wordpress.org
Fri Jul 22 00:41:09 UTC 2022


#22176: Cache the results of the posts_request_ids query
-------------------------------------------------+-------------------------
 Reporter:  ryan                                 |       Owner:
                                                 |  spacedmonkey
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  6.1
Component:  Query                                |     Version:  3.4.2
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch 2nd-opinion has-unit-      |     Focuses:
  tests early needs-testing needs-dev-note       |  performance
-------------------------------------------------+-------------------------

Comment (by peterwilsoncc):

 I could be convinced either way but am leaning to on by default from the
 initial commit:

 * reduces the complexity of the change
 * avoids a new argument that will eventually have the same default as the
 existing, similarly named one
 * Dion makes a good point about not using existing WP APIs requiring an
 acceptance of risk (which can be mitigated by flushing the `last_changed`
 cache)
 * the change needs a dev-note either way

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


More information about the wp-trac mailing list