summaryrefslogtreecommitdiff
path: root/mysql-test/suite/rpl
Commit message (Collapse)AuthorAgeFilesLines
* MDEV-21921 Make transaction_isolation and transaction_read_only into system ↵Junqi Xie2023-04-127-11/+11
| | | | | | | | | | | | | | variables In MariaDB, we have a confusing problem where: * The transaction_isolation option can be set in a configuration file, but it cannot be set dynamically. * The tx_isolation system variable can be set dynamically, but it cannot be set in a configuration file. Therefore, we have two different names for the same thing in different contexts. This is needlessly confusing, and it complicates the documentation. The same thing applys for transaction_read_only. MySQL 5.7 solved this problem by making them into system variables. https://dev.mysql.com/doc/relnotes/mysql/5.7/en/news-5-7-20.html This commit takes a similar approach by adding new system variables and marking the original ones as deprecated. This commit also resolves some legacy problems related to SET STATEMENT and transaction_isolation.
* Merge 10.11 into 11.0Marko Mäkelä2023-03-2952-2825/+2803
|\
| * Merge 10.10 into 10.11Marko Mäkelä2023-03-2950-2823/+2801
| |\
| | * Merge 10.9 into 10.10Marko Mäkelä2023-03-2950-2823/+2801
| | |\
| | | * Merge 10.8 into 10.9Marko Mäkelä2023-03-2950-2823/+2801
| | | |\
| | | | * Merge 10.6 into 10.8Marko Mäkelä2023-03-2950-2823/+2801
| | | | |\
| | | | | * Merge 10.5 into 10.6Marko Mäkelä2023-03-2842-2809/+2787
| | | | | |\
| | | | | | * MDEV-30421 rpl_parallel_*.test cleanupbb-10.5-midenokAleksey Midenkov2023-03-2342-2797/+2776
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Moved rpl_parallel_*.inc to rpl_parallel_*.test
| | | | | * | Merge 10.5 into 10.6Marko Mäkelä2023-03-228-14/+14
| | | | | |\ \ | | | | | | |/
| | | | | | * [MDEV-30824] Fix binlog to use 'String' for setting 'character_set_client'Tingyao Nian2023-03-218-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit a923d6f49c1ad6fd3f4d6ec02e444c26e4d1dfa8 disabled numeric setting of character_set_* variables with non-default values: MariaDB [(none)]> set character_set_client=224; ERROR 1115 (42000): Unknown character set: '224' However the corresponding binlog functionality still write numeric values for log event, and this will break binlog replay if the value is not default. Now make the server use 'String' type for 'character_set_client' when generating binlog events Before: /*!\C utf8mb4 *//*!*/; SET @@session.character_set_client=224,@@session.collation_connection=224,@@session.collation_server=33/*!*/; After: /*!\C utf8mb4 *//*!*/; SET @@session.character_set_client=utf8mb4,@@session.collation_connection=33,@@session.collation_server=8/*!*/; Note: prior to the previous commit, setting with '224' or '45' or 'utf8mb4' have the same effect, as they all set the parameter to 'utf8mb4'. All new code of the whole pull request, including one or several files that are either new files or modified ones, are contributed under the BSD-new license. I am contributing on behalf of my employer Amazon Web Services, Inc.
| * | | | | | Fix trivial spelling errorsOtto Kekalainen2023-03-242-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - agressively -> aggressively - exising -> existing - occured -> occurred - releated -> related - seperated -> separated - sucess -> success - use use -> use All new code of the whole pull request, including one or several files that are either new files or modified ones, are contributed under the BSD-new license. I am contributing on behalf of my employer Amazon Web Services, Inc.
* | | | | | | Merge 10.11 into 11.0Marko Mäkelä2023-02-1651-159/+1204
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | Merge 10.10 into 10.11Marko Mäkelä2023-02-1638-159/+73
| |\ \ \ \ \ \ | | |/ / / / /
| | * | | | | Merge 10.9 into 10.10Marko Mäkelä2023-02-162-0/+29
| | |\ \ \ \ \ | | | |/ / / /
| | | * | | | Merge 10.8 into 10.9Marko Mäkelä2023-02-162-0/+29
| | | |\ \ \ \ | | | | |/ / /
| | | | * | | Fix RPL tests post DEBUG_SYNC changeVicențiu Ciorbaru2023-02-101-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tests expect the SIGNAL to not be cleared. Thus set NO_CLEAR_EVENT within DBUG_EXECUTE_IF
| | | | * | | Merge 10.6 into 10.8Marko Mäkelä2023-02-103-0/+40
| | | | |\ \ \ | | | | | |/ /
| | | | | * | Merge 10.5 into 10.6Marko Mäkelä2023-02-102-0/+29
| | | | | |\ \ | | | | | | |/
| | | | | | * Merge 10.4 into 10.5Marko Mäkelä2023-02-102-0/+29
| | | | | | |\
| | | | | | | * MDEV-30608: rpl.rpl_delayed_parallel_slave_sbm sometimes fails with ↵Brandon Nesterenko2023-02-092-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Seconds_Behind_Master should not have used second transaction timestamp One of the constraints added in the MDEV-29639 patch, is that only the first event after idling should update last_master_timestamp; and as long as the replica has more events to execute, the variable should not be updated. The corresponding test, rpl_delayed_parallel_slave_sbm.test, aims to verify this; however, if the IO thread takes too long to queue events, the SQL thread can appear to catch up too fast. This fix ensures that the relay log has been fully written before executing the events. Note that the underlying cause of this test failure needs to be addressed as a bug-fix, this is a temporary fix to stop test failures. To track work on the bug-fix for the underlying issue, please see MDEV-30619.
| | | | | | | * Merge branch '10.3' into 10.4Oleksandr Byelkin2023-01-286-0/+429
| | | | | | | |\
| | * | | | | | | MDEV-28839: remove current_pos where not intentionally being testedasklavou2023-02-1336-159/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Task: ===== Update tests to reflect MDEV-20122, deprecation of master_use_gtid=current_pos. Change Master (CM) statements were either removed or modified with current_pos --> slave_pos based on original intention of the test. Reviewed by: ============ Brandon Nesterenko <brandon.nesterenko@mariadb.com>
| * | | | | | | | Merge branch '10.10' into 10.11Oleksandr Byelkin2023-01-3115-0/+1131
| |\ \ \ \ \ \ \ \ | | |/ / / / / / /
| | * | | | | | | Merge branch '10.9' into 10.10Oleksandr Byelkin2023-01-3115-0/+1131
| | |\ \ \ \ \ \ \ | | | |/ / / / / /
| | | * | | | | | Merge branch '10.8' into 10.9Oleksandr Byelkin2023-01-3115-0/+1131
| | | |\ \ \ \ \ \ | | | | |/ / / / /
| | | | * | | | | Merge branch '10.7' into 10.8Oleksandr Byelkin2023-01-3115-0/+1131
| | | | |\ \ \ \ \
| | | | | * \ \ \ \ Merge branch '10.6' into 10.7Oleksandr Byelkin2023-01-3115-0/+1131
| | | | | |\ \ \ \ \ | | | | | | |/ / / /
| | | | | | * | | | Merge branch '10.5' into 10.6Oleksandr Byelkin2023-01-3115-0/+1131
| | | | | | |\ \ \ \ | | | | | | | |/ / /
| | | | | | | * | | Merge branch '10.4' into 10.5Oleksandr Byelkin2023-01-276-0/+429
| | | | | | | |\ \ \
| | | | | | | | * \ \ Merge branch '10.3' into 10.4Oleksandr Byelkin2023-01-266-0/+429
| | | | | | | | |\ \ \ | | | | | | | | | |/ / | | | | | | | | |/| / | | | | | | | | | |/
| | | | | | | | | * MDEV-30323 Some DDLs like ANALYZE can complete on parallel slave out of orderbb-10.3-andreiAndrei2023-01-243-0/+234
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ANALYZE was observed to race over a preceding in binlog order DML in updating the binlog and slave gtid states. Tagging ANALYZE and other admin class commands in binlog by the fixes of MDEV-17515 left a flaw allowing such race leading to the gtid mode out-of-order error. This is fixed now to observe by ADMIN commands the ordered access to the slave gtid status variables and binlog.
| | | | | | | | | * MDEV-29639: Seconds_Behind_Master is incorrect for Delayed, Parallel ReplicasBrandon Nesterenko2023-01-243-0/+194
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem ======== On a parallel, delayed replica, Seconds_Behind_Master will not be calculated until after MASTER_DELAY seconds have passed and the event has finished executing, resulting in potentially very large values of Seconds_Behind_Master (which could be much larger than the MASTER_DELAY parameter) for the entire duration the event is delayed. This contradicts the documented MASTER_DELAY behavior, which specifies how many seconds to withhold replicated events from execution. Solution ======== After a parallel replica idles, the first event after idling should immediately update last_master_timestamp with the time that it began execution on the primary. Reviewed By =========== Andrei Elkin <andrei.elkin@mariadb.com>
| | | | | | | * | | MDEV-30423 Deadlock on Replica during BACKUP STAGE BLOCK_COMMIT on XA ↵Andrei2023-01-239-0/+700
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | transactions The user XA commit execution branch was caught not have been covered with MDEV-21953 fixes. The XA involved deadlock is resolved now to apply the former fixes pattern. Along the fixes the following changes have been implemented. - MDL lock attribute correction - dissociation of the externally completed XA from the current thread's xid_state in the error branches - cleanup_context() preseves the prepared XA - wait_for_prior_commit() is relocated to satisfy both the binlog ON (log-slave-updates and skip-log-bin) and OFF slave execution branches.
* | | | | | | | | | Merge 11.0-selectivity into 11.0Sergei Petrunia2023-02-159-30/+8
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Removed "<select expression> INTO <destination>" deprication.Monty2023-02-035-22/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was done after discussions with Igor, Sanja and Bar. The main reason for removing the deprication was to ensure that MariaDB is always backward compatible whenever possible. Other things: - Added statistics counters, mainly for the feedback plugin. - INTO OUTFILE - INTO variable - If INTO is using the old syntax (end of query)
| * | | | | | | | | | Removed diff dates from rdiff filesMonty2023-02-034-8/+8
| |/ / / / / / / / /
* | | | | | | | | | MDEV-30498 Rename mysql_upgrade state file to mariadb_upgradeChristian Gonzalez2023-02-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Renames the upgrade state file, and ensures the old file is properly removed when `mariadb-upgrade` tool is executed. All new code of the whole pull request, including one or several files that are either new files or modified ones, are contributed under the BSD-new license. I am contributing on behalf of my employer Amazon Web Services, Inc.
* | | | | | | | | | MDEV-29668 SUPER should not allow actions that have fine-grained dedicated ↵Sergei Golubchik2023-02-062-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | privileges SUPER privilege used to allow various actions that were alternatively allowed by one of BINLOG ADMIN, BINLOG MONITOR, BINLOG REPLAY, CONNECTION ADMIN, FEDERATED ADMIN, REPL MASTER ADMIN, REPL SLAVE ADMIN, SET USER, SLAVE MONITOR. Now SUPER no longer does that, one has to grant one of the fine-grained privileges above to be to perform corresponding actions. On upgrade from MariaDB versions 10.11 and below all the privileges above are granted automatically if the user has SUPER. As a side-effect, such an upgrade will allow SUPER-user to run SHOW BINLOG EVENTS, SHOW RELAYLOG EVENTS, SHOW SLAVE HOSTS, even if he wasn't able to do it before the upgrade.
* | | | | | | | | | MDEV-29227 deprecate explicit_defaults_for_timestamp=0Sergei Golubchik2023-02-053-0/+8
| | | | | | | | | |
* | | | | | | | | | MDEV-30128 remove support for 5.1- replication eventsSergei Golubchik2023-02-053-71/+0
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | including patches from Andrei Elkin
* | | | | | | | | Merge 10.10 into 10.11Marko Mäkelä2022-12-143-4/+6
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / /
| * | | | | | | | Merge 10.9 into 10.10Marko Mäkelä2022-12-143-4/+6
| |\ \ \ \ \ \ \ \ | | |/ / / / / / /
| | * | | | | | | Merge 10.8 into 10.9Marko Mäkelä2022-12-133-4/+6
| | |\ \ \ \ \ \ \ | | | |/ / / / / /
| | | * | | | | | Merge 10.7 into 10.8Marko Mäkelä2022-12-133-4/+6
| | | |\ \ \ \ \ \ | | | | |/ / / / /
| | | | * | | | | Merge 10.6 into 10.7Marko Mäkelä2022-12-133-4/+6
| | | | |\ \ \ \ \ | | | | | |/ / / /
| | | | | * | | | Merge 10.5 into 10.6Marko Mäkelä2022-12-133-4/+6
| | | | | |\ \ \ \ | | | | | | |/ / /
| | | | | | * | | Merge 10.4 into 10.5Marko Mäkelä2022-12-133-4/+6
| | | | | | |\ \ \ | | | | | | | |/ /
| | | | | | | * | Merge 10.3 into 10.4Marko Mäkelä2022-12-133-4/+6
| | | | | | | |\ \ | | | | | | | | |/
| | | | | | | | * MDEV-30056 Impossible to export column grantsSergei Golubchik2022-12-021-4/+4
| | | | | | | | |
| | | | | | | | * Enable valgrind for replication testMonty2022-11-292-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following tests are disabled when running --valgrding without --big: - rpl.rpl_ssl - rpl.rpl_semi_sync_event - All encryption test (which includes have_file_key_management.inc)