Added FIXME to default Database replace() implementation
authorAaron Schulz <aschulz@wikimedia.org>
Fri, 17 Jul 2015 22:22:54 +0000 (15:22 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Fri, 17 Jul 2015 22:22:54 +0000 (15:22 -0700)
Change-Id: I3ec0ec066248ee733b41f9c70d40382d96898613

includes/db/Database.php

index 2f1155d..04b3edd 100644 (file)
@@ -2873,6 +2873,7 @@ abstract class DatabaseBase implements IDatabase {
                        $rows = array( $rows );
                }
 
+               // @FXIME: this is not atomic, but a trx would break affectedRows()
                foreach ( $rows as $row ) {
                        # Delete rows which collide
                        if ( $uniqueIndexes ) {