Aaron Schulz has uploaded a new change for review. https://gerrit.wikimedia.org/r/316399
Change subject: Make isTransactableQuery() exclude CREATE/ALTER ...................................................................... Make isTransactableQuery() exclude CREATE/ALTER Starting transactions for schema changes like this can cause errors as it is not supported for MySQL and some Postgres operations. Note that temporary tables are session-level, so they are not effected by this change. Bug: T147599 Change-Id: I64ccab7f9b74f60309ba0c9a8ce68337c42ffb0f --- M includes/libs/rdbms/database/Database.php 1 file changed, 5 insertions(+), 2 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core refs/changes/99/316399/1 diff --git a/includes/libs/rdbms/database/Database.php b/includes/libs/rdbms/database/Database.php index a5a170b..db24b43 100644 --- a/includes/libs/rdbms/database/Database.php +++ b/includes/libs/rdbms/database/Database.php @@ -773,8 +773,11 @@ * @return bool */ protected function isTransactableQuery( $sql ) { - $verb = $this->getQueryVerb( $sql ); - return !in_array( $verb, [ 'BEGIN', 'COMMIT', 'ROLLBACK', 'SHOW', 'SET' ], true ); + return !in_array( + $this->getQueryVerb( $sql ), + [ 'BEGIN', 'COMMIT', 'ROLLBACK', 'SHOW', 'SET', 'CREATE', 'ALTER' ], + true + ); } /** -- To view, visit https://gerrit.wikimedia.org/r/316399 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I64ccab7f9b74f60309ba0c9a8ce68337c42ffb0f Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Aaron Schulz <asch...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits