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

Timestamp:

11/20/2019 06:59:20 AM (4 years ago)

Author:

dd32

Message:

Plugin Directory: Cron Jobs: Clear the Memcache cache within WordPress after processing lots of posts to allow Jetpack Sync to have some memory free.

This avoids plenty of post-shutdown fatals in cron tasks WordPress.org is currently experiencing and consolidates such code to a single instance.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/jobs/class-meta-sync.php (modified) (3 diffs)

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

@@ -26 +26 @@
         $this->sync_ratings();
         $this->update_tested_up_to();
+
+        Manager::clear_memory_heavy_variables();
     }
 
@@ -167 +169 @@
         $tested_values             = $wpdb->get_results( "SELECT post_id, post_name, meta_value FROM {$wpdb->postmeta} pm JOIN {$wpdb->posts} p ON pm.post_id = p.ID WHERE meta_key = 'tested' AND meta_value IN( {$tested_meta_value_esc_sql} )" );
 
-        foreach ( $tested_values as $row ) {
+        foreach ( $tested_values as $i => $row ) {
             update_post_meta(
                 $row->post_id,
@@ -176 +178 @@
             // Update the API endpoints with the new data
             API_Update_Updater::update_single_plugin( $row->post_name );
+
+            if ( $i % 100 === 0 ) {
+                Manager::clear_memory_heavy_variables();
+            }
         }
     }