Merge "Do not let DataUpdate exceptions block other deferred updates"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Fri, 6 May 2016 09:55:27 +0000 (09:55 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Fri, 6 May 2016 09:55:27 +0000 (09:55 +0000)
includes/deferred/DeferredUpdates.php

index 249b207..e3b7570 100644 (file)
@@ -144,7 +144,12 @@ class DeferredUpdates {
                        }
 
                        // Delegate DataUpdate execution to the DataUpdate class
-                       DataUpdate::runUpdates( $dataUpdates, $mode );
+                       try {
+                               DataUpdate::runUpdates( $dataUpdates, $mode );
+                       } catch ( Exception $e ) {
+                               // Let the other updates occur if these had to rollback
+                               MWExceptionHandler::logException( $e );
+                       }
                        // Execute the non-DataUpdate tasks
                        foreach ( $otherUpdates as $update ) {
                                try {