rdbms: remove HIGH_PRIORITY/LOW_PRIORITY SQL references from IDatabase
authorAaron Schulz <aschulz@wikimedia.org>
Wed, 7 Aug 2019 20:44:06 +0000 (13:44 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Wed, 7 Aug 2019 22:04:53 +0000 (15:04 -0700)
These should not be used even with MySQL as they do not help concurrency

Change-Id: I5385fcbe65e676b184268286bedd5c2299b447d4

includes/libs/rdbms/database/Database.php
includes/libs/rdbms/database/DatabaseMssql.php
includes/libs/rdbms/database/DatabaseMysqlBase.php
includes/libs/rdbms/database/IDatabase.php

index ffd4026..8b4ff67 100644 (file)
@@ -1715,10 +1715,6 @@ abstract class Database implements IDatabase, IMaintainableDatabase, LoggerAware
                        $startOpts .= ' /*! STRAIGHT_JOIN */';
                }
 
-               if ( isset( $noKeyOptions['HIGH_PRIORITY'] ) ) {
-                       $startOpts .= ' HIGH_PRIORITY';
-               }
-
                if ( isset( $noKeyOptions['SQL_BIG_RESULT'] ) ) {
                        $startOpts .= ' SQL_BIG_RESULT';
                }
index db029a3..9d4a364 100644 (file)
@@ -785,7 +785,6 @@ class DatabaseMssql extends Database {
         *
         * @param array $options An array of UPDATE options, can be:
         *                   - IGNORE: Ignore unique key conflicts
-        *                   - LOW_PRIORITY: MySQL-specific, see MySQL manual.
         * @return bool
         * @throws DBUnexpectedError
         * @throws Exception
index b1a88ed..a9223ac 100644 (file)
@@ -1085,13 +1085,6 @@ abstract class DatabaseMysqlBase extends Database {
                return "IGNORE INDEX (" . $this->indexName( $index ) . ")";
        }
 
-       /**
-        * @return string
-        */
-       function lowPriorityOption() {
-               return 'LOW_PRIORITY';
-       }
-
        /**
         * @return string
         */
index f66e327..1c42d2d 100644 (file)
@@ -709,7 +709,6 @@ interface IDatabase {
         *
         *    - LOCK IN SHARE MODE
         *    - STRAIGHT_JOIN
-        *    - HIGH_PRIORITY
         *    - SQL_BIG_RESULT
         *    - SQL_BUFFER_RESULT
         *    - SQL_SMALL_RESULT
@@ -717,7 +716,6 @@ interface IDatabase {
         *    - SQL_CACHE
         *    - SQL_NO_CACHE
         *
-        *
         * @param string|array $join_conds Join conditions
         *
         * Optional associative array of table-specific join conditions. In the
@@ -929,7 +927,6 @@ interface IDatabase {
         *   for logging and profiling.
         * @param array $options An array of UPDATE options, can be:
         *   - IGNORE: Ignore unique key conflicts
-        *   - LOW_PRIORITY: MySQL-specific, see MySQL manual.
         * @return bool Return true if no exception was thrown (deprecated since 1.33)
         * @throws DBError
         */