[wp-trac] [WordPress Trac] #36723: Add caching to wp_old_slug_redirect

[WordPress Trac]

#36723: Add caching to wp_old_slug_redirect
-------------------------------------------------+-------------------------
 Reporter:  spacedmonkey                         |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
Component:  Posts, Post Types                    |  Review
 Severity:  normal                               |     Version:  2.1
 Keywords:  has-patch dev-feedback needs-unit-   |  Resolution:
  tests                                          |     Focuses:
                                                 |  performance
-------------------------------------------------+-------------------------

Comment (by spacedmonkey):

 Sorry @dd32 I am not sure what you mean by "fake" page. The patch just
 caches the response from sql. That is then invalided when the
 clean_post_cache function is called and the last_changed value changes. I
 know WordPress VIP are doing something similar with caching the result of
 this function. Still not sure why this needs to call sql on every time.
 You could easily have a popular url that is 404ing as you remove the post.
 This just increase traffic to sql unnecessarily.

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