summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix GCC 10.2.0 -Og -Wmaybe-uninitializedMarko Mäkelä2020-08-1111-76/+76
| | | | | | For some reason, GCC emits more -Wmaybe-uninitialized warnings when using the flag -Og than when using -O2. Many of the warnings look genuine.
* MDEV-22543 : Galera SST donation fails, FLUSH TABLES WITH READ LOCK times outJan Lindström2020-08-115-9/+106
| | | | | During SST we need to let FTWRL to use normal timeout method even when client is disconnected.
* MDEV-23378 - fix an alleged memory "leak" in threadpool.Vladislav Vaintroub2020-08-101-0/+11
| | | | Implement a workaround to shut the "memory not freed" message.
* Merge 10.1 into 10.2Marko Mäkelä2020-08-1019-410/+412
|\
| * MDEV-16115 Hang after reducing innodb_encryption_threadsMarko Mäkelä2020-08-106-374/+222
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test encryption.create_or_replace would occasionally fail, because some fil_space_t::n_pending_ops would never be decremented. fil_crypt_find_space_to_rotate(): If rotate_thread_t::should_shutdown() holds due to innodb_encryption_threads having been reduced, do release the reference. fil_space_remove_from_keyrotation(), fil_space_next(): Declare the functions static, simplify a little, and define in the same compilation unit with the only caller, fil_crypt_find_space_to_rotate(). fil_crypt_key_mutex: Remove (unused).
| * bump the VERSIONDaniel Bartholomew2020-08-101-1/+1
| |
| * Merge remote-tracking branch 'bb-10.1-release' into 10.1Oleksandr Byelkin2020-08-101-1/+1
| |\
| * | MDEV-23386: mtr: main.mysqld--help autosized table{-open,}-cach and ↵Daniel Black2020-08-105-31/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | max-connections Example of the failure: http://buildbot.askmonty.org/buildbot/builders/bld-p9-rhel7/builds/4417/steps/mtr/logs/stdio ``` main.mysqld--help 'unix' w17 [ fail ] Test ended at 2020-06-20 18:51:45 CURRENT_TEST: main.mysqld--help --- /opt/buildbot-slave/bld-p9-rhel7/build/mysql-test/main/mysqld--help.result 2020-06-20 16:06:49.903604179 +0300 +++ /opt/buildbot-slave/bld-p9-rhel7/build/mysql-test/main/mysqld--help.reject 2020-06-20 18:51:44.886766820 +0300 @@ -1797,10 +1797,10 @@ sync-relay-log-info 10000 sysdate-is-now FALSE system-versioning-alter-history ERROR -table-cache 421 +table-cache 2000 table-definition-cache 400 -table-open-cache 421 -table-open-cache-instances 1 +table-open-cache 2000 +table-open-cache-instances 8 tc-heuristic-recover OFF tcp-keepalive-interval 0 tcp-keepalive-probes 0 mysqltest: Result length mismatch ``` mtr: table_open_cache_basic autosized: Lets assume that >400 are available and that we can set the result back to the start value. All of these system variables are autosized and can generate MTR output differences. Closes #1527
| * | Better comment about TABLE::maybe_nullSergei Petrunia2020-08-061-3/+10
| | |
| * | MDEV-9513: Assertion `join->group_list || !join->is_in_subquery()' failed in ↵Varun Gupta2020-08-064-0/+89
| | | | | | | | | | | | | | | | | | | | | | | | create_sort_index Removing the ORDER BY clause from the UNION when UNION is inside an IN/ALL/ANY/EXISTS subquery. The rewrites are done for subqueries but this rewrite is not done for the fake_select of the UNION.
| * | MDEV-23105 Cast number string with many leading zeros to decimal gives ↵Alexander Barkov2020-08-053-0/+84
| | | | | | | | | | | | | | | | | | unexpected result Skip leading zeros when converting a string to decimal_t.
* | | Merge mariadb-10.2.33Marko Mäkelä2020-08-1047-107/+126
|\ \ \
| * \ \ Merge branch '10.1' into 10.2mariadb-10.2.33Sergei Golubchik2020-08-0647-107/+126
| |\ \ \ | | | |/ | | |/|
| | * | link failure on fulltest (xenial)mariadb-10.1.46Sergei Golubchik2020-08-061-1/+1
| | |/ | | | | | | | | | | | | | | | depending on build config the error might be hidded, in particular liblz4.so and libjemalloc.so make it to disappear, but with -DWITH_INNODB_LZ4=NO -DWITH_JEMALLOC=NO it reappears.
| | * Merge branch 'merge/merge-tokudb-5.6' into 10.1Sergei Golubchik2020-08-0542-100/+95
| | |\
| | | * 5.6.49-89.0Sergei Golubchik2020-08-0447-148/+125
| | | |
| | * | MDEV-23089 rpl_parallel2 fails in 10.5Sachin2020-08-043-4/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem:- rpl_parallel2 was failing non-deterministically Analysis:- When FLUSH TABLES WITH READ LOCK is executed, it will allow all worker threads to complete their ongoing transactions and then it will pause them. At this state FTWRL will proceed to acquire global read lock. FTWRL first blocks threads from starting new commits, then upgrades the lock to block commit of existing transactions. Step1: FLUSH TABLES WITH READ LOCK - Blocks new commits Step2: * STOP SLAVE command enables 'force_abort=1' which unblocks workers, they continue to execute events. * T1: Waits in 'record_gtid' call to update 'gtid_slave_pos' table with its current GTID, but it is blocked becuase of Step1. * T2: Holds COMMIT lock and waits for T1 to commit. Step3: FLUSH TABLES WITH READ LOCK - Waiting to get BLOCK_COMMIT. This results in deadlock. When STOP SLAVE command allows paused workers to proceed, workers should skip the execution of all further events, similar to 'conservative' parallel mode. Solution:- We will assign 1 to skip_event_group when we are aborted in do_ftwrl_wait. rpl_parallel_entry->pause_sub_id is only reset when force_abort is off in rpl_pause_after_ftwrl.
| | * | Fix the typo in fix for MDEV-21472Sergei Petrunia2020-08-031-1/+1
| | | |
| | * | compilation error on bintar-centos6-amd64-debugSergei Golubchik2020-08-031-2/+1
| | | | | | | | | | | | | | | | /home/buildbot/buildbot/build/storage/xtradb/mtr/mtr0mtr.cc:97:37: error: invalid access to non-static data member ‘fil_space_t::latch’ of NULL object [-Werror=invalid-offsetof]
* | | | bump the VERSIONDaniel Bartholomew2020-08-101-1/+1
| | | |
* | | | Replaced infinite loop in procedure with limited loop to avoidbb-10.2-MDEV-22626Jan Lindström2020-08-072-17/+67
| | | | | | | | | | | | | | | | hang.
* | | | MDEV-22626: mysql_tzinfo_to_sql not replicates timezone to galeranodes if ↵Jan Lindström2020-08-073-9/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | only 1 timezone will be loaded. Move alter to InnoDB earlier to more correct place to handle also if only a one timezone file is loaded.
* | | | MDEV-15180: server crashed with NTH_VALUE()Varun Gupta2020-08-063-1/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | fix_fields for the arguments of the NTH_VALUE function was updating the same reference, so for the second argument (or after the first argument) the items were not resolved to their corresponding field from the view as they were updating the reference to the first argument.
* | | | MDEV-17066: Bytes lost or Assertion `status_var.local_memory_used == 0 after ↵Varun Gupta2020-08-056-25/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DELETE with subquery with ROLLUP The issue here is when records are read from the temporary file (filesort result in this case) via a cache(rr_from_cache). The cache is initialized with init_rr_cache. For correlated subquery the cache allocation is happening at each execution of the subquery but the deallocation happens only once and that was when the query execution was done. So generally for subqueries we do two types of cleanup 1) Full cleanup: we should free all resources of the query(like temp tables). This is done generally when the query execution is complete or the subquery re-execution is not needed (case with uncorrelated subquery) 2) Partial cleanup: Minor cleanup that is required if the subquery needs recalculation. This is done for all the structures that need to be allocated for each execution (example SORT_INFO for filesort is allocated for each execution of the correlated subquery). The fix here would be free the cache used by rr_from_cache in the partial cleanup phase.
* | | | MDEV-23101 fixup: Remove redundant codeMarko Mäkelä2020-08-041-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | lock_rec_has_to_wait_in_queue(): Remove an obviously redundant assertion that was added in commit a8ec45863b958757da61af3b2ce0a38b0a79d92c and also enclose a Galera-specific condition in #ifdef WITH_WSREP.
* | | | MDEV-23082: ER_TABLEACCESS_DENIED_ERROR error message is truncated,Rucha Deodhar2020-08-043-19/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and inaccurately Analysis: The list of all privileges is 118 characters wide. However, the format of error message was: "%-.32s command denied to user...". get_length() sets the maximum width to 32 characters. As a result, only first 32 characters of list of privilege are stored. Fix: Changing the format to "%-.100T..." so that get_length() sets width to 100. Hence, first 100 characters of the list of privilege are stored and the type specifier 'T' appends '...' so that truncation can be seen.
* | | | MDEV-14836: Assertion `m_status == DA_ERROR' failed inRucha Deodhar2020-08-043-6/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Diagnostics_area::sql_errno upon query from I_S with LIMIT ROWS EXAMINED open_normal_and_derived_table() fails because the query was already killed as rows examined by the query are more than the limit. However, this isn't a real error. Fix: Check if there is actually an error before calling thd->sql_errno() and later send a warning in handle_select() if no real error.
* | | | Test case MW-328A still fails, thus disable it until it is reallyJan Lindström2020-08-041-0/+1
| | | | | | | | | | | | | | | | fixed.
* | | | MDEV-23101: SIGSEGV in lock_rec_unlock() when Galera is enabledJan Lindström2020-08-031-10/+11
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | lock_rec_has_to_wait wsrep_kill_victim lock_rec_create_low lock_rec_add_to_queue DeadlockChecker::select_victim() THD can't change from normal transaction to BF (brute force) transaction here, thus there is no need to syncronize access in wsrep_thd_is_BF function. lock_rec_has_to_wait_in_queue Add condition that lock is not NULL and add assertions if we are in strong state.
* | | List of unstable tests for 10.2.33 releaseElena Stepanova2020-08-031-325/+268
| | |
* | | Merge branch '10.1' into 10.2Elena Stepanova2020-08-032-0/+5
|\ \ \ | |/ /
| * | MDEV-23375 parts.partition_debug fails when it's run after another testElena Stepanova2020-08-032-0/+5
| | | | | | | | | | | | Make sure system tables aren't open, as the test kills the server
| * | List of unstable tests for 10.1.46 releaseElena Stepanova2020-08-021-120/+132
| | |
* | | Fix for macOleksandr Byelkin2020-08-021-1/+4
| | |
* | | C/C v3.1.9Oleksandr Byelkin2020-08-021-0/+0
| | |
* | | Merge remote-tracking branch 'connect/10.2' into 10.2Oleksandr Byelkin2020-08-0217-102/+505
|\ \ \
| * | | Fix compile error for KVM. Modified filamdbf.cppOlivier Bertrand2020-07-131-1/+1
| | | |
| * | | - Fix MDEV-22561 Unable to access DBF inside a ZIP archiveOlivier Bertrand2020-07-1314-100/+490
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | modified: storage/connect/filamdbf.cpp modified: storage/connect/filamdbf.h modified: storage/connect/filamzip.cpp modified: storage/connect/filamzip.h modified: storage/connect/ha_connect.cc modified: storage/connect/plgxml.cpp modified: storage/connect/tabdos.cpp modified: storage/connect/tabdos.h modified: storage/connect/tabfix.h - Add/Init Level class member modified: storage/connect/mongo.cpp modified: storage/connect/mongo.h modified: storage/connect/tabjson.cpp modified: storage/connect/tabjson.h - Typo modified: storage/connect/connect.cc
| * | | - Fix MDEV-22571 and MDEV-22572. Allow multiple ZIP tableOlivier Bertrand2020-05-193-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and enable using special column in them. modified: storage/connect/tabzip.cpp modified: storage/connect/tabzip.h - Fix some compiler errors modified: storage/connect/tabcmg.cpp
* | | | Merge branch '10.1' into 10.2Oleksandr Byelkin2020-08-0298-398/+964
|\ \ \ \ | | |/ / | |/| |
| * | | improve the error message for a dropped current roleSergei Golubchik2020-07-303-5/+7
| | | |
| * | | MDEV-22521 Server crashes in traverse_role_graph_up or Assertion `user' ↵Sergei Golubchik2020-07-303-0/+16
| | | | | | | | | | | | | | | | fails in traverse_role_graph_impl
| * | | XtraDB 5.6.49-89.0Marko Mäkelä2020-07-303-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only change between Percona XtraDB Server 5.6.48-88.0 and 5.6.49-89.0 (apart from the version number change) was percona/percona-server@25ec24092064c2ab95752705e592e0c038ec1111 which we had already addressed in commit 7c03edf2fe66855a8ce8f2575c3aaf66af975377 and commit c0fca2863bcbd7cd231f1aa747b4f8d999e3a00e.
| * | | MDEV-21472: ALTER TABLE ... ANALYZE PARTITION ... with EITS reads and locks ↵Sergei Petrunia2020-07-294-4/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | all rows Do not collect EITS statistics for this statement: ALTER TABLE t ANALYZE PARTITION p EITS stats are currently global, not per-partition. Collecting global stats when we are asked to process just one partition causes issues for DBAs.
| * | | MDEV-23010 UPDATE privilege at Database and Table level fail to update with ↵Sergei Golubchik2020-07-293-3/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | SELECT command denied to user check both column- and table-level grants when looking for SELECT privilege on UPDATE statement.
| * | | cleanup: reduce code duplicationSergei Golubchik2020-07-293-45/+21
| | | |
| * | | Bug #25207522: INCORRECT ORDER-BY BEHAVIOR ON A PARTITIONED TABLE WITH A ↵Sergei Golubchik2020-07-295-24/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | COMPOSITE PREFIX INDEX Fix prefix key comparison in partitioning. Comparions must take into account no more than prefix_len characters. It used to compare prefix_len*mbmaxlen bytes.
| * | | bugfix: mysql_create_view() infinite loopSergei Golubchik2020-07-291-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | if mysql_create_view() is aborted when view is linked into lex (when WSREP_TO_ISOLATION_BEGIN fails), it should not be linked there again on err:.
| * | | MDEV-18496 Crash when Aria encryption is enabled but plugin not availableSergei Golubchik2020-07-293-0/+8
| | | | | | | | | | | | | | | | fix uninitialized struct member
| * | | MDEV-18496 Crash when Aria encryption is enabled but plugin not availableSergei Golubchik2020-07-294-2/+24
| | | | | | | | | | | | | | | | wait_while_table_is_used() should return an error if handler::extra() fails