1. 17 Feb, 2021 1 commit
  2. 16 Feb, 2021 1 commit
  3. 14 Feb, 2021 4 commits
    • Monty's avatar
      d0defd1e
    • Monty's avatar
      MDEV-23843 Assertions in Diagnostics_area upon table operations under FTWRL · af31e2c5
      Monty authored
      2 different problems:
      - MYSQL_BIN_LOG::write() did not check if mdl_context.acquire_lock() failed
      - Sql_cmd_optimize_table::execute() and Sql_cmd_repair_table::execute()
        called write_bin_log(), which could fail if sql_admin() had already
        called my_eof()
      
      Fixed by adding check for aquire_lock() return status and protect
      write_bin_log() in the above two functions with set_overwrite_status().
      af31e2c5
    • Monty's avatar
      Remove extra (not needed) error from multi-table-update for killed query · 3cd32a9b
      Monty authored
      If a multi-update query was killed because of timeout or a kill statement,
      there was a chance the query returned the error "An error occurred in
      multi-table update" instead of the right error, like
      "Query execution was interrupted".
      3cd32a9b
    • Vlad Lesin's avatar
      MDEV-24792 Assertion `!newest_lsn || fil_page_get_type(page)' failed upon... · 23833dce
      Vlad Lesin authored
      MDEV-24792 Assertion `!newest_lsn || fil_page_get_type(page)' failed upon MariaBackup prepare in buf_flush_init_for_writing with innodb_log_optimize_ddl=off
      
      fsp_free_page() writes MLOG_INIT_FREE_PAGE, but does not update page
      type. But fil_crypt_rotate_page() checks the type to understand if the
      page is freshly initialized, and writes dummy record(updates space id)
      to force rotation during recovery. This dummy record causes assertion
      crash when the page is flushed after recovery, as it's supposed
      that pages LSN is 0 for freshly initialized pages.
      
      The bug is similiar to MDEV-24695, the difference is that in 10.5 the
      assertion crashes during log record applying, but in 10.4 it crashes
      during page flushing.
      
      The fix could be in marking page as freed and not writing dummy record
      during keys rotation procedure for such marked pages. But
      bpage->file_page_was_freed is not consistent enough for release builds in
      10.4, and the issue is fixed in 10.5 and does not exist in 10.[23] as
      MLOG_INIT_FREE_PAGE was introduced since 10.4.
      
      So the better solution is just to relax the assertion and implement some
      additional property for freshly allocated pages, and check this property
      during pages flushing.
      
      The test is copied from MDEV-24695, the only change is in forcing pages
      flushing after each server start to cause crash in non-fixed code.
      
      There is no need to merge it to 10.5+, as the bug is already fixed by
      MDEV-24695.
      23833dce
  4. 13 Feb, 2021 1 commit
  5. 28 Jan, 2021 1 commit
  6. 27 Jan, 2021 1 commit
  7. 26 Jan, 2021 1 commit
  8. 25 Jan, 2021 6 commits
  9. 22 Jan, 2021 2 commits
  10. 21 Jan, 2021 3 commits
    • Sergei Golubchik's avatar
      MDEV-24593 Signal 11 when group by primary key of table joined to information_schema.columns · 4e503aec
      Sergei Golubchik authored
      I_S tables were materialized too late, an attempt to use table
      statistics before the table was created caused a crash.
      
      Let's move table creation up. it only needs read_set to
      be calculated properly, this happens in JOIN::optimize_inner(),
      after semijoin transformation.
      
      Note that tables are not populated at that point, so most of the
      statistics would make no sense anyway. But at least field sizes
      will be correct. And it won't crash.
      4e503aec
    • Sergei Golubchik's avatar
      remove now-unused rdiff file · 61feb568
      Sergei Golubchik authored
      61feb568
    • Jan Lindström's avatar
      MDEV-24596 : Assertion `state_ == s_exec || state_ == s_quitting' failed in... · be5fce16
      Jan Lindström authored
      MDEV-24596 : Assertion `state_ == s_exec || state_ == s_quitting' failed in wsrep::client_state::disable_streaming
      
      There were multiple problems here
      * wsrep_trx_fragment_size should not be set when wsrep is disabled or provider is not loaded
      * wsrep_trx_fragment_unit should not be set when wsrep is disabled or provider is not loaded
      * wsrep_debug has no effect if wsrep is disabled or provider is not loaded
      * wsrep_start_position should not be set when wsrep is disabled or provider is not loaded any other value than default
      * wsrep_start_position should be changed only when we are joiner or initialized
      * wsrep_start_position should be allowed to set only a value that exits, thus
      we need to add error handling to wsrep_sst_complete
      be5fce16
  11. 20 Jan, 2021 1 commit
    • sjaakola's avatar
      MDEV-21153 Replica nodes crash due to indexed virtual columns and FK cascading delete · 9377e9ba
      sjaakola authored
      Fix for MDEV-23033 fixes a problem in replication applying of transactions, which contain cascading foreign key delete for a table, which has indexed virtual column.
      This fix adds slave_fk_event_map flag for table, to mark when the prelocking is needed for applying of a transaction.
      See commit 608b0ee5 for more details.
      However, this fix is targeted for async replication only, Rows_log_event::do_apply_event() has condition to rule out galera replication from the fix domain, and use cases suffering from MDEV-23033 and related MDEV-21153 will fail in galera cluster.
      
      The fix in this commit removes the condition to rule out the setting of slave_fk_event_map flag from galera replication, and makes the fix in MDEV-23033 effective for galera replication as well.
      
      However, the above fix has caused regressions for some galera_sr suite tests, which run tests for streaming replication.
      This regression can be observed e.g. by: /mtr galera_sr.galera_sr_multirow_rollback  --mysqld=--slave_run_triggers_for_rbr=yes
      These galera_sr suite tests were failing in last phase of replication applying, where actual transaction is already applied, and streaming replication related meta data needs to be updated in wsrep system tables.
      Opening the wsrep system tables failed for corrupt data in THD::lex:query_tables_list. The fix in this commit uses back query table list for the duration of fragment update operation.
      
      Finally, a mtr test for virtual column support has been added. galera.galera_virtual_column.test has as first test a scenario from MDEV-21153
      
      new fix
      Reviewed-by: default avatarJan Lindström <jan.lindstrom@mariadb.com>
      9377e9ba
  12. 19 Jan, 2021 3 commits
    • sjaakola's avatar
      MDEV-21153 Replica nodes crash due to indexed virtual columns and FK cascading delete · 7d04ce6a
      sjaakola authored
      Fix for MDEV-23033 fixes a problem in replication applying of transactions, which contain cascading foreign key delete for a table, which has indexed virtual column.
      This fix adds slave_fk_event_map flag for table, to mark when the prelocking is needed for applying of a transaction.
      See commit 608b0ee5 for more details.
      However, this fix is targeted for async replication only, Rows_log_event::do_apply_event() has condition to rule out galera replication from the fix domain, and use cases suffering from MDEV-23033 and related MDEV-21153 will fail in galera cluster.
      
      The fix in this commit removes the condition to rule out the setting of slave_fk_event_map flag from galera replication, and makes the fix in MDEV-23033 effective for galera replication as well.
      
      Finally, a mtr test for virtual column support has been added. galera.galera_virtual_column.test has as first test a scenario from MDEV-21153
      Reviewed-by: default avatarJan Lindström <jan.lindstrom@mariadb.com>
      7d04ce6a
    • Dmitry Shulga's avatar
      MDEV-24577: Fix warnings generated during compilation of... · 8bcddb02
      Dmitry Shulga authored
      MDEV-24577: Fix warnings generated during compilation of plugin/auth_pam/testing/pam_mariadb_mtr.c on FreeBSD
      
      Compiler warnings generated on building MariaDB server for BSD has the same
      reason as in case building is performed on MacOS. Both platforms do use
      clang as a C/C++ compiler. So, fix the compiler warnings in case the compiler
      is clang doesn't matter what kind of building platform do we use for building.
      
      This is a follow-up patch for the following bug reports:
        MDEV-23564: CMAKE failing due to deprecated Apple GSS method
        MDEV-23935: Fix warnings generated during compilation of
                    plugin/auth_pam/testing/pam_mariadb_mtr.c on MacOS
      8bcddb02
    • Marko Mäkelä's avatar
      Merge 10.2 into 10.3 · 049811ec
      Marko Mäkelä authored
      049811ec
  13. 14 Jan, 2021 2 commits
    • Jan Lindström's avatar
      MDEV-22285 : Assertion `xid_seqno > wsrep_seqno' failed in... · b87828b6
      Jan Lindström authored
      MDEV-22285 : Assertion `xid_seqno > wsrep_seqno' failed in trx_rseg_update_wsrep_checkpoint on SET @@global.wsrep_start_position
      
      Actual assertion mentioned on MDEV seems to be already fixed but
      setting seqno to -2 will trigger a different assertion
      
      mysqld: /home/jan/mysql/10.4-bugs/wsrep-lib/src/server_state.cpp:702: void wsrep::server_state::sst_received(wsrep::client_service&, int): Assertion `state_ == s_joiner || state_ == s_initialized' failed.
      
      Fixed this by not allowing user to set seqno < -1 (-1 is special
      seqno meaning undefined and seqno is initialized to it). MariaDB
      releases 10.2 and 10.3 already do not allow to set seqno < -1.
      b87828b6
    • Dmitry Shulga's avatar
      MDEV-23666: Assertion `m_cpp_buf <= ptr && ptr <= m_cpp_buf + m_buf_length'... · f130adbf
      Dmitry Shulga authored
      MDEV-23666: Assertion `m_cpp_buf <= ptr && ptr <= m_cpp_buf + m_buf_length' failed in Lex_input_stream::body_utf8_append
      
      On parsing statements for which a starting backtick (`) delimiter doesn't have
      a corresponding ending backtick, a current pointer to a position inside a
      pre-processed buffer could go beyond the end of the buffer.
      
      This bug report caused by the commit d4967659
        "MDEV-22022 Various mangled SQL statements will crash 10.3 to 10.5 debug builds".
      
      In order to fix the issue both pointers m_ptr and m_cpp_ptr must be
      rolled back to previous position in raw input and pre-processed input streams
      correspondingly in case end of query reached during parsing.
      f130adbf
  14. 13 Jan, 2021 3 commits
  15. 12 Jan, 2021 10 commits