Changeset 19918 for trunk/wp-includes/query.php

Timestamp:

02/14/2012 03:09:35 PM (13 years ago)

Author:

ryan

Message:

Split the main WP_Query posts query into two queries to avoid temp tables. Leverage cache to avoid second query in persistent cache environments. Props scribu, cheald, prettyboymp. see #18536

File:

• trunk/wp-includes/query.php (modified) (4 diffs)

trunk/wp-includes/query.php

@@ -2607 +2607 @@
             $found_rows = 'SQL_CALC_FOUND_ROWS';
 
-        $this->request = " SELECT $found_rows $distinct $fields FROM $wpdb->posts $join WHERE 1=1 $where $groupby $orderby $limits";
-        if ( !$q['suppress_filters'] )
-            $this->request = apply_filters_ref_array('posts_request', array( $this->request, &$this ) );
+        $this->request = $old_request = "SELECT $found_rows $distinct $fields FROM $wpdb->posts $join WHERE 1=1 $where $groupby $orderby $limits";
+
+        if ( !$q['suppress_filters'] ) {
+            $this->request = apply_filters( 'posts_request', $this->request, $this );
+        }
 
         if ( 'ids' == $q['fields'] ) {
@@ -2627 +2629 @@
         }
 
-        $this->posts = $wpdb->get_results($this->request);
+        if ( $old_request == $this->request && "$wpdb->posts.*" == $fields ) {
+            // First get the IDs and then fill in the objects
+
+            $this->request = "SELECT $found_rows $distinct $wpdb->posts.ID FROM $wpdb->posts $join WHERE 1=1 $where $groupby $orderby $limits";
+
+            $this->request = apply_filters( 'posts_request_ids', $this->request, $this );
+
+            $ids = $wpdb->get_col( $this->request );
+
+            if ( $ids ) {
+                $this->set_found_posts( $q, $limits );
+
+                _prime_post_caches( $ids, $q['update_post_term_cache'], $q['update_post_meta_cache'] );
+
+                $this->posts = array_map( 'get_post', $ids );
+            } else {
+                $this->found_posts = $this->max_num_pages = 0;
+                $this->posts = array();
+            }
+        } else {
+            $this->posts = $wpdb->get_results( $this->request );
+            $this->set_found_posts( $q, $limits );
+        }
 
         // Raw results filter. Prior to status checks.
@@ -2644 +2668 @@
             $this->comments = $wpdb->get_results($comments_request);
             $this->comment_count = count($this->comments);
-        }
-
-        if ( !$q['no_found_rows'] && !empty($limits) ) {
-            $found_posts_query = apply_filters_ref_array( 'found_posts_query', array( 'SELECT FOUND_ROWS()', &$this ) );
-            $this->found_posts = $wpdb->get_var( $found_posts_query );
-            $this->found_posts = apply_filters_ref_array( 'found_posts', array( $this->found_posts, &$this ) );
-            $this->max_num_pages = ceil($this->found_posts / $q['posts_per_page']);
         }
 
@@ -2755 +2772 @@
     }
 
+    function set_found_posts( $q, $limits ) {
+        global $wpdb;
+
+        if ( $q['no_found_rows'] || empty( $limits ) )
+            return;
+
+        $this->found_posts = $wpdb->get_var( apply_filters( 'found_posts_query', 'SELECT FOUND_ROWS()', $this ) );
+        $this->found_posts = apply_filters( 'found_posts', $this->found_posts, $this );
+
+        $this->max_num_pages = ceil( $this->found_posts / $q['posts_per_page'] );
+    }
+
     /**
      * Set up the next post and iterate current post index.