Changeset 5017 for sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/official-wordpress-events.php

Timestamp:

02/28/2017 08:29:05 PM (9 years ago)

Author:

iandunn

Message:

Official WordPress Events: Add temporary logging to debug stuck cron jobs

Sometimes this cron job gets stuck in a running state in Cavalcade. It might be that the process is dying and the status never gets set to completed or failed. Hopefully this provides some clues.

See ​https://github.com/humanmade/Cavalcade/issues/31

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/official-wordpress-events.php (modified) (10 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/official-wordpress-events.php

@@ -56 +56 @@
         global $wpdb;
 
+        $this->log( 'started call #' . did_action( 'owpe_prime_events_cache' ) );
+
+        if ( did_action( 'owpe_prime_events_cache' ) > 1 ) {
+            $this->log( 'Successive call detected, returning early' );
+            return;
+        }
+
         $events = $this->fetch_upcoming_events();
+
+        $this->log( sprintf( 'looping through %d events', count( $events ) ) );
 
         foreach ( $events as $event ) {
@@ -92 +101 @@
             $wpdb->replace( self::EVENTS_TABLE, $row_values );
         }
+
+        $this->log( 'finished job' );
     }
 
@@ -246 +257 @@
         }
 
+        $this->log( sprintf( 'returning %d events', count( $events ) ) );
+
         return $events;
     }
@@ -407 +420 @@
                 $response = $this->remote_get( $request_url );
                 $body     = json_decode( wp_remote_retrieve_body( $response ) );
+
+                $this->log( 'fetching more events' );
 
                 if ( ! empty ( $body->results ) ) {
@@ -458 +473 @@
         }
 
+        $this->log( sprintf( 'returning %d events', count( $events ) ) );
+
         return $events;
     }
@@ -481 +498 @@
 
         while ( '' !== $request_url ) {
+            $this->log( 'fetching more groups' );
+
             $response = $this->remote_get( $request_url );
             $body     = json_decode( wp_remote_retrieve_body( $response ) );
@@ -491 +510 @@
             $request_url = $body->meta->next;
         }
+
+        $this->log( sprintf( 'returning %d groups', count( $group_ids ) ) );
 
         return $group_ids;
@@ -512 +533 @@
 
         if ( ! is_wp_error( $response ) ) {
+            $this->log( 'geocode successful' );
+
             $body = json_decode( wp_remote_retrieve_body( $response ) );
 
@@ -517 +540 @@
                 $address = $body->results[0];
             }
+        }
+        else {
+            $this->log( 'geocode failed' );
         }
 
@@ -654 +680 @@
         }
 
+        $this->log( 'sleeping to avoid api rate limit' );
         sleep( $period );
+    }
+
+    /**
+     * Log messages to the database
+     *
+     * To avoid storing too much data, the log is reset during each run, and only $limit rows are stored
+     *
+     * @param string $message
+     */
+    protected function log( $message ) {
+        $limit = 500;
+
+        if ( ! isset( $this->log ) ) {
+            $this->log = array();
+        }
+
+        if ( count( $this->log ) > $limit ) {
+            wp_die('early return');
+            return;
+        }
+
+        $backtrace = debug_backtrace( DEBUG_BACKTRACE_PROVIDE_OBJECT, 2 );
+
+        $this->log[] = sprintf(
+            '%s - %s MB - %s - %s',
+            microtime( true ),
+            number_format( memory_get_usage( true ) / 1024 / 2014, 2 ),
+            $backtrace[1]['function'],
+            $message
+        );
+
+        if ( $limit === count( $this->log ) ) {
+            $this->log[] = array( 'Reached log limit, assuming some kind of infinite loop. Will not log any more messages.' );
+        }
+
+        update_option( 'owpe_log', $this->log, false );
     }
 }