Changeset 16137

Timestamp:

11/01/2010 05:53:03 PM (14 years ago)

Author:

ryan

Message:

For large networks, show the latest registered blogs without paging to avoid expensive count queries. see #15170

File:

• trunk/wp-admin/includes/class-wp-ms-sites-list-table.php (modified) (2 diffs)

trunk/wp-admin/includes/class-wp-ms-sites-list-table.php

@@ -32 +32 @@
         $s = isset( $_REQUEST['s'] ) ? stripslashes( trim( $_REQUEST[ 's' ] ) ) : '';
         $like_s = esc_sql( like_escape( $s ) );
+
+        $large_network = false;
+        // If the network is large and a search is not being performed, show only the latest blogs with no paging in order
+        // to avoid expensive count queries.
+        if ( !$s && ( get_blog_count() >= 10000 ) ) {
+            if ( !isset($_REQUEST['orderby']) )
+                $_GET['orderby'] = $_REQUEST['orderby'] = 'registered';
+            if ( !isset($_REQUEST['order']) )
+                $_GET['order'] = $_REQUEST['order'] = 'DESC';
+            $large_network = true;
+        }
 
         $query = "SELECT * FROM {$wpdb->blogs} WHERE site_id = '{$wpdb->siteid}' ";
@@ -64 +75 @@
         $query .= $order;
 
-        $total = $wpdb->get_var( str_replace( 'SELECT *', 'SELECT COUNT( blog_id )', $query ) );
+        // Don't do an unbounded count on large networks
+        if ( ! $large_network )
+            $total = $wpdb->get_var( str_replace( 'SELECT *', 'SELECT COUNT( blog_id )', $query ) );
 
         $query .= " LIMIT " . intval( ( $pagenum - 1 ) * $per_page ) . ", " . intval( $per_page );
         $this->items = $wpdb->get_results( $query, ARRAY_A );
+
+        if ( $large_network )
+            $total = count($this->items);
 
         $this->set_pagination_args( array(