Make DeferredUpdates enqueue updates that failed to run when possible

Bug: T206288
Bug: T206283
Change-Id: I6025bcc7d68cf214d291191d4044a66cdeff108b

diff --git a/includes/deferred/DeferredUpdates.php b/includes/deferred/DeferredUpdates.php
index 67b5490..a14b25c 100644 (file)
--- a/includes/deferred/DeferredUpdates.php
+++ b/includes/deferred/DeferredUpdates.php
@@ -280,6 +280,16 @@ class DeferredUpdates {
                        }
                        MWExceptionHandler::rollbackMasterChangesAndLog( $e );
 
+                       // Try to push the update as a job so it can run later perhaps
+                       if ( $mode !== 'enqueue' && $update instanceof EnqueueableDataUpdate ) {
+                               try {
+                                       $spec = $update->getAsJobSpecification();
+                                       JobQueueGroup::singleton( $spec['wiki'] )->push( $spec['job'] );
+                               } catch ( Exception $e ) {
+                                       MWExceptionHandler::rollbackMasterChangesAndLog( $e );
+                               }
+                       }
+
                        // VW-style hack to work around T190178, so we can make sure
                        // PageMetaDataUpdater doesn't throw exceptions.
                        if ( defined( 'MW_PHPUNIT_TEST' ) ) {