Changeset 10230

Timestamp:

09/02/2020 04:41:58 AM (6 years ago)

Author:

dd32

Message:

Plugin Directory: Jobs: Meta Sync: Run the sync twice hourly, and only sync download counts that have changed more than 10.

This is a performance boost for the ElasticSearch syncing which can have many thousands of plugins updating the download count by 1 every hour, which slows down indexing for everything else.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/jobs

Files:

• class-manager.php (modified) (1 diff)
• class-meta-sync.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/jobs/class-manager.php

@@ -241 +241 @@
         public function register_cron_tasks() {
                 if ( ! wp_next_scheduled( 'plugin_directory_meta_sync' ) ) {
-                        wp_schedule_event( time() + 60, 'hourly', 'plugin_directory_meta_sync' );
+                        wp_schedule_event( time() + 60, 'half_hourly', 'plugin_directory_meta_sync' );
                 }
                 if ( ! wp_next_scheduled( 'plugin_directory_plugin_support_resolved' ) ) {

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/jobs/class-meta-sync.php

@@ -41 +41 @@
                         "SELECT p.id as post_id, downloads
                         FROM `{$wpdb->posts}` p
-                                JOIN `{$download_count_table}` c on p.post_name = c.plugin_slug
+                                JOIN `{$download_count_table}` c ON p.post_name = c.plugin_slug
                                 LEFT JOIN `{$wpdb->postmeta}` pm ON p.id = pm.post_id AND pm.meta_key = 'downloads'
 
                         WHERE
-                                downloads != pm.meta_value OR
-                                pm.meta_id IS NULL"
+                                pm.meta_id IS NULL
+                                OR
+                                downloads - pm.meta_value > 10
+                        "
                 );