[wp-trac] [WordPress Trac] #15499: Add an index for get_lastpostmodified query

WordPress Trac noreply at wordpress.org
Thu Jan 29 06:39:39 UTC 2015


#15499: Add an index for get_lastpostmodified query
-------------------------------------------------+-------------------------
 Reporter:  simonwheatley                        |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Future
Component:  Database                             |  Release
 Severity:  normal                               |     Version:  3.0.1
 Keywords:  has-patch dev-feedback 4.1-early     |  Resolution:
  commit                                         |     Focuses:
                                                 |  performance
-------------------------------------------------+-------------------------

Comment (by archon810):

 Replying to [comment:22 Denis-de-Bernardy]:
 > There's another salient ticket over at #4575. The need for a last
 modified date is based on the feed, and what could be done instead of
 adding an index is this:
 >
 > https://core.trac.wordpress.org/ticket/4575#comment:14
 >
 > In essence, rather than serving '''the''' absolute last modified, we
 could fetch the posts or comments that appear in a feed, and then simply
 loop through the array to find ''their'' last modified date (which would
 then indeed concern that specific feed, and that only, solving this ticket
 and #4575 in one go).

 If feeds are the only users of this unoptimized function, and by resolving
 that ticket you can remove deprecate this function, then by all means,
 I'll be glad to remove the unnecessary index. However, if it's only going
 to solve it for feeds but other places in the code may still use it, then
 the index should be put in place.

 Either way, hopefully this saga comes to its end in 4.2.

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


More information about the wp-trac mailing list