summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* mysys: remove windac my_security_attr_create (#1391)Daniel Black2020-02-143-210/+0
| | | No longer used.
* micro optimization: avoid std::string copyEugene Kosov2020-02-131-2/+3
|
* MDEV-21669 InnoDB: Table ... contains <n> indexes inside InnoDB, which is ↵Eugene Kosov2020-02-135-5/+33
| | | | | | | | | | | | | | | | | | | | different from the number of indexes <n> defined in the MariaDB compare_keys_but_name(): do not use KEY_PART_INFO::field for Field::is_equal(). Following the logic of that code we need to compare fields of a table. But KEY_PART_INFO::field sometimes (when key part is shorter than table field) is a different field. In that case Field::is_equal() returns incorrect result and problems occur. KEY_PART_INFO::field may become some strange field in open_frm_error open_table_from_share(). I think this is an incorrect logic, some tecnhical debt. I'm not fixing it right now, because I don't have time. But I'm making Field::field_length a const class member. Then, the only fishy code which changed that field requires now a const_cast<>. I'm bringing attention to that code with it. This change should not affect logic of the program in any way.
* MDEV-20867 - Perform careful review of "Server crashes with BACKUP STAGE and ↵Sergey Vojtovich2020-02-135-53/+16
| | | | | | | | | | | | | FLUSH TABLE table_name" Reverted original patch (c2e0a0b). For consistency with "LOCK TABLE <table_name> READ" and "FLUSH TABLES WITH READ LOCK", which are forbidden under "BACKUP STAGE", forbid "FLUSH TABLE <table_name> FOR EXPORT" and "FLUSH TABLE <table_name> WITH READ LOCK" as well. It'd allow consistent fixes for problems like MDEV-18643.
* Merge branch '10.3' into 10.4Oleksandr Byelkin2020-02-11164-2282/+2854
|\
| * Merge branch '10.2' into 10.3Oleksandr Byelkin2020-02-1023-90/+129
| |\
| | * Ignore /lib64 for rpmAnel Husakovic2020-02-101-0/+2
| | |
| | * Merge branch '10.1' into 10.2Oleksandr Byelkin2020-02-107-7/+53
| | |\
| | | * Merge branch '5.5' into 10.1Oleksandr Byelkin2020-02-101-1/+0
| | | |\
| | | | * Clean the comment for `table_f_c unt` parameterAnel Husakovic2020-01-291-1/+0
| | | | | | | | | | | | | | | | | | | | Deleted with commit: c70a9fa1e3c4
| | | | * bump the VERSIONDaniel Bartholomew2020-01-271-1/+1
| | | | |
| | | * | Remove unused SRV_MASTER_PURGE_INTERVALMarko Mäkelä2020-02-082-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The symbol SRV_MASTER_PURGE_INTERVAL became unused in mysql/mysql-server@42f36919584e82c621dbec1e69fd05ab023c54c6 when separate purge threads were introduced in MySQL 5.6.5.
| | | * | MDEV-21563 FTS thread aborts during shutdownThirunarayanan Balathandayuthapani2020-02-074-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | | - Added the test case in innodb_fts suite - Updated copyright year in row0mysql.cc
| | | * | MDEV-21563 FTS thread aborts during shutdownThirunarayanan Balathandayuthapani2020-02-062-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ======= After discarding the table, fts_optimize_thread aborts during shutdown. InnoDB fails to remove the table from fts_optimize_wq and it leads to the fts_optimize_thread to lookup for the auxiliary table and fails. Fix: ==== While discarding the fts table, remove the table from fts_optimize_wq.
| | * | | MDEV-16308 : protocol messed up sporadicallyVladislav Vaintroub2020-02-101-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Context involves semicolon batching, and the error starts 10.2 No reproducible examples were made yet, but TCP trace suggests multiple packets that are "squeezed" together (e.g overlong OK packet that has a trailer which is belongs to another packet) Remove thd->get_stmt_da()->set_skip_flush() when processing a batch. skip_flush stems from the COM_MULTI code, which was checked in during 10.2 (and is never used) The fix is confirmed to work, when evaluated by bug reporter (one of them) We never reproduced it locally, with multiple tries thus the root cause analysis is still missing.
| | * | | MDEV-21667 : Galera test failure on MW-336Jan Lindström2020-02-092-24/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem seems to be the fact that we did not enforce correct applier thread numbers after every command that effects them. Test changes only.
| | * | | MDEV-21601 : Cleanup Galera disabled testsJan Lindström2020-02-0920-1244/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove those tests that will not be supported on that release. * Make sure that correct tests are disabled and have MDEVs * Sort test names This should not be merged upwards.
| | * | | added warning to ignoreOleksandr Byelkin2020-02-071-2/+2
| | | | |
| | * | | Windows test fixOleksandr Byelkin2020-02-071-13/+14
| | | | |
| | * | | MDEV-21608 Assertion `n_ext == dtuple_get_n_ext(dtuple)' failed during ↵Thirunarayanan Balathandayuthapani2020-02-073-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | updation of PK - n_ext value may be less than dtuple_get_n_ext(dtuple) when PK is being updated and new record inherits the externally stored fields from delete mark old record.
| | * | | MDEV-12121: Clean up WITH_INNODB_AHI=OFFMarko Mäkelä2020-02-061-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | buf_flush_or_remove_pages(): Only define BUF_LRU_DROP_SEARCH_SIZE and dependent code when the adaptive hash index has been enabled.
| | * | | MDEV-21656: Wrong directory for pam_user_map.soAurélien LEQUOY2020-02-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | JIRA:https://jira.mariadb.org/browse/MDEV-17292 Closes #1443
| | * | | MDEV-18027: Running out of file descriptors and eventual crashOleksandr Byelkin2020-02-053-5/+13
| | | | | | | | | | | | | | | | | | | | For automatic number of opened files limit take into account number of table instances for table cache
| | * | | Cleanup: Remove mem_block_t::magic_n and mem_block_validate()Marko Mäkelä2020-02-033-46/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use of freed memory is better caught by AddressSanitizer, especially with ASAN_POISON_MEMORY_REGION that is aliased by MEM_NOACCESS and UNIV_MEM_FREE.
| | * | | MDEV-21636 information_schema.innodb_mutexes.name column is not populatedMarko Mäkelä2020-02-031-8/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The column INFORMATION_SCHEMA.INNODB_MUTEXES.NAME is not populated ever since commit 2e814d4702d71a04388386a9f591d14a35980bfe applied the InnoDB changes from MySQL 5.7.9 to MariaDB Server 10.2.2. Since the same commit, the view is only providing information about rw_lock_t, not any mutexes. For now, let us convert the source code file name and line number of the rw_lock_t creation into a name. A better option in the future might be to store the information somewhere where it can be looked up by mysql_pfs_key_t, and possibly to remove the CREATE_FILE and CREATE_LINE columns.
| | * | | introduce HASH_REPLACE() for hash_table_tEugene Kosov2020-01-312-4/+16
| | | | | | | | | | | | | | | | | | | | | | | | | HASH_REPLACE(): allows to not travel through linked list twice when HASH_INSERT() happens right after HASH_DELETE()
| * | | | MDEV-21667 : Galera test failure on MW-336Jan Lindström2020-02-092-24/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem seems to be the fact that we did not enforce correct applier thread numbers after every command that effects them. Test changes only.
| * | | | MDEV-21601 : Cleanup Galera disabled testsJan Lindström2020-02-0920-1244/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove those tests that will not be supported on that release. * Make sure that correct tests are disabled and have MDEVs * Sort test names This should not be merged upwards.
| * | | | MDEV-21614 Wrong query results with optimizer_switch="split_materialized=on"Igor Babaev2020-02-073-1/+142
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not materialize a semi-join nest if it contains a materialized derived table /view that potentially can be subject to the split optimization. Splitting of materialization of such nest would help, but currently there is no code to support this technique.
| * | | | MDEV-20076: SHOW GRANTS does not quote role names properlybb-10.3-MDEV-20076Oleksandr Byelkin2020-02-05103-1997/+2124
| | | | | | | | | | | | | | | | | | | | Quotes added to output.
| * | | | ha_partition: add comments, comment out unused member variablesSergei Petrunia2020-02-052-3/+90
| | | | |
| * | | | MDEV-21195 INSERT chooses wrong partition for RANGE partitioning by DECIMAL ↵Aleksey Midenkov2020-02-025-6/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | column Use FLOOR rounding for DECIMAL_RESULT item_expr in partition function.
| * | | | MDEV-21317 mysqlhotcopy and transaction_registry tableAleksey Midenkov2020-02-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See also original report: http://bugs.debian.org/946671 Using mysqlhotcopy, the following error occurs: DBD::mysql::db do failed: You can't use locks with log tables at /usr/bin/mysqlhotcopy line 545. Author: Paul Szabo psz@maths.usyd.edu.au http://www.maths.usyd.edu.au/u/psz/ School of Mathematics and Statistics University of Sydney Australia
| * | | | MDEV-20528 innodb.purge_secondary_mdev-16222 failed in buildbot, debug sync ↵Aleksey Midenkov2020-02-022-3/+5
| | | | | | | | | | | | | | | | | | | | point wait timed out
| * | | | MDEV-17798 System variable system_versioning_asof accepts wrong valuesAleksey Midenkov2020-02-023-2/+34
| | | | |
| * | | | MDEV-20955 versioning.update failed in buildbot with wrong result codeAleksey Midenkov2020-02-022-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Race condition when innodb_lock_wait_timeout (default 50 seconds) exceeds for 'send update', but information_schema.innodb_lock_waits still sees this wait or it my exit by timeout. My occur on overloaded host.
| * | | | MDEV-14330: After-merge fixMarko Mäkelä2020-01-311-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The merge commit 5ff66fb0b97fa3f8ecc453c48bac48ceae76bdea accidentally omitted part of commit 07e34cddb66da2e9e4ab5bdd8d52d1a72c2d2e8e.
| * | | | Merge 10.2 into 10.3Marko Mäkelä2020-01-3118-93/+100
| |\ \ \ \ | | |/ / /
| | * | | MDEV-21586: Fix a warning for converting my_bool to boolMarko Mäkelä2020-01-311-3/+4
| | | | |
| | * | | Merge 10.1 into 10.2Marko Mäkelä2020-01-319-129/+68
| | |\ \ \ | | | |/ /
| | | * | MDEV-20307: Remove a useless debug check to save stack spaceMarko Mäkelä2020-01-312-84/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fil_space_encrypt(): Remove the debug check that decrypts the just encrypted page. We are exercising the decryption of encrypted pages enough via --suite=encryption,mariabackup. It is a waste of computing resources to decrypt every page immediately after encrypting it. The redundant check had been added in commit 2bedc3978b90bf5abe1029df393c63ced1849bed (MDEV-9931).
| | | * | MDEV-21564 Assert in trx_purge_add_update_undo_to_history during shutdownThirunarayanan Balathandayuthapani2020-01-302-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | - dict_table_try_drop_aborted() can generate undo logs. So it should be avoided after shutdown of background threads.
| | | * | MDEV-20923:UBSAN: member access within address … which does not point to ↵Sujatha2020-01-292-17/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | an object of type 'xid_count_per_binlog' Problem: ------- Accessing a member within 'xid_count_per_binlog' structure results in following error when 'UBSAN' is enabled. member access within address 0xXXX which does not point to an object of type 'xid_count_per_binlog' Analysis: --------- The problem appears to be that no constructor for 'xid_count_per_binlog' is being called, and thus the vtable will not be initialized. Fix: --- Defined a parameterized constructor for 'xid_count_per_binlog' class.
| | | * | MDEV-14330 Move mysqltest.1 man page to appropriate test package from server ↵Daniel Black2020-01-281-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | package Original patch from Daniel Black <daniel@linux.ibm.com>, backported to 10.1.
| | | * | Ingore sysusers and tmpfiles artifactsVicențiu Ciorbaru2020-01-281-0/+2
| | | | |
| | | * | Merge branch 'bb-10.1-release' into 10.1Oleksandr Byelkin2020-01-281-146/+117
| | | |\ \
| | | | * | List of unstable tests for 10.1.44 releasemariadb-10.1.44Elena Stepanova2020-01-251-146/+117
| | | | | |
| | | * | | bump the VERSIONDaniel Bartholomew2020-01-271-1/+1
| | | | | |
| | | * | | MDEV-15052: Allow sysusers and tmpfiles install for non-systemd usersRafli Akmal2020-01-241-17/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ..as they have their own tools that parses those files, such as opensysusers[1] that handles sysusers file and opentmpfiles[2] that handles tmpfiles.d settings Because of this. Move both sysusers and tmpfiles 'if' function outside systemd function, allowing independent install Signed-off-by: Rafli Akmal <thefallenrat@artixlinux.org> [1] - https://github.com/artix-linux/opensysusers [2] - https://github.com/OpenRC/opentmpfiles Changes done by vicentiu@mariadb.org, from original author patch: Installing sysusers and tmpfiles without checking for systemd existence means that by default, cmake will ALWAYS install these files. Our general policy is we do not install things which are not needed. However, there is a valid use case when these files are useful, as is described above. To allow this, provide an extra switch that can be enabled during configuring by doing -DINSTALL_SYSTEMD_{SYSUSERS|TMPFILES}=True This will use the default path INSTALL_SYSTEMD_{SYSUSERS|TMPFILES}DIR fetched from install_layout.cmake for rpm & deb based layouts respectively, or they must be overriden if the install_layout is standalone. Example: cmake . -DINSTALL_SYSTEMD_SYSUSERS=True -DINSTALL_SYSTEMD_SYSUSERSDIR=/etc/sysusers.d
| | | * | | MDEV-17028: Use descriptive file names for sysusers and tmpfiles configurationChristian Hesse2020-01-241-2/+4
| | | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These files were installed to: ${INSTALL_SYSTEMD_SYSUSERSDIR}/sysusers.conf ${INSTALL_SYSTEMD_TMPFILESDIR}/tmpfiles.conf Instead rename the files to more descriptive file names 'mariadb.conf'.