Changeset 10230

Timestamp:

09/02/2020 04:41:58 AM (5 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
+            "
         );