1. 07 May, 2019 3 commits
  2. 04 May, 2019 1 commit
    • Sergey Vojtovich's avatar
      Restore vars_list destructor · e8dd18a4
      Sergey Vojtovich authored
      Regression after reverting fair THD members constructors/destructors.
      vars_list can be used standalone, in such cases destructor is needed.
      
      Part of MDEV-14984 - regression in connect performance
      e8dd18a4
  3. 03 May, 2019 18 commits
    • Marko Mäkelä's avatar
      Merge 10.2 into 10.3 · b6f4cccd
      Marko Mäkelä authored
      b6f4cccd
    • Marko Mäkelä's avatar
      MDEV-19385: Inconsistent definition of dtuple_get_nth_v_field() · ce195987
      Marko Mäkelä authored
      The accessor dtuple_get_nth_v_field() was defined differently between
      debug and release builds in MySQL 5.7.8 in
      mysql/mysql-server@c47e1751b742454de553937039bbf2bcbe3c6bc7
      and a debug assertion to document or enforce the questionable assumption
      tuple->v_fields == &tuple->fields[tuple->n_fields] was missing.
      
      This was apparently no problem until MDEV-11369 introduced instant
      ADD COLUMN to MariaDB Server 10.3. With that work present, in one
      test case, trx_undo_report_insert_virtual() could in release builds
      fetch the wrong value for a virtual column.
      
      We replace many of the dtuple_t accessors with const-preserving
      inline functions, and fix missing or misleadingly applied const
      qualifiers accordingly.
      ce195987
    • Marko Mäkelä's avatar
      MDEV-19346: Remove dummy InnoDB log checkpoints · 3db94d24
      Marko Mäkelä authored
      log_checkpoint(), log_make_checkpoint_at(): Remove the parameter
      write_always. It seems that the primary purpose of this parameter
      was to ensure in the function recv_reset_logs() that both checkpoint
      header pages will be overwritten, when the function is called from
      the never-enabled function recv_recovery_from_archive_start().
      
      create_log_files(): Merge recv_reset_logs() to its only caller.
      
      Debug instrumentation: Prefer to flush the redo log, instead of
      triggering a redo log checkpoint.
      
      page_header_set_field(): Disable a debug assertion that will
      always fail due to MDEV-19344, now that we no longer initiate
      a redo log checkpoint before an injected crash.
      
      In recv_reset_logs() there used to be two calls to
      log_make_checkpoint_at(). The apparent purpose of this was
      to ensure that both InnoDB redo log checkpoint header pages
      will be initialized or overwritten.
      The second call was removed (without any explanation) in MySQL 5.6.3:
      mysql/mysql-server@4ca37968da54ddc6b3b6628f41428ddba1c79bb8
      
      In MySQL 5.6.8 WL#6494, starting with
      mysql/mysql-server@00a0ba8ad92569fcf08212b3b8cf046dc8a0ce10
      the function recv_reset_logs() was not only invoked during
      InnoDB data file initialization, but also during a regular
      startup when the redo log is being resized.
      
      mysql/mysql-server@45e91679832219e2593c77185342f11f85232b58
      in MySQL 5.7.2 removed the UNIV_LOG_ARCHIVE code, but still
      did not remove the parameter write_always.
      3db94d24
    • Eugene Kosov's avatar
      MDEV-17702 fix unaligned access UB in sint4korr() and similar functions · bcc13592
      Eugene Kosov authored
      Disable (hopefully temprorary) this check. Also add tests for some serialized
      functions.
      bcc13592
    • Sergey Vojtovich's avatar
      Revert THD::THD(skip_global_sys_var_lock) argument · 779fb636
      Sergey Vojtovich authored
      Originally introduced by e972125f to avoid harmless wait for
      LOCK_global_system_variables in a newly created thread, which creation was
      initiated by system variable update.
      
      At the same time it opens dangerous hole, when system variable update
      thread already released LOCK_global_system_variables and ack_receiver
      thread haven't yet completed new THD construction. In this case THD
      constructor goes completely unprotected.
      
      Since ack_receiver.stop() waits for the thread to go down, we have to
      temporarily release LOCK_global_system_variables so that it doesn't
      deadlock with ack_receiver.run(). Unfortunately it breaks atomicity
      of rpl_semi_sync_master_enabled updates and makes them not serialized.
      
      LOCK_rpl_semi_sync_master_enabled was introduced to workaround the above.
      TODO: move ack_receiver start/stop into repl_semisync_master
      enable_master/disable_master under LOCK_binlog protection?
      
      Part of MDEV-14984 - regression in connect performance
      779fb636
    • Sergey Vojtovich's avatar
      Adieu find_sys_var_ex() · 894df7ed
      Sergey Vojtovich authored
      Only take LOCK_plugin for plugin system variables.
      
      Reverted optimisation that was originally done for session tracker: it
      makes much less sense now. Specifically only if connections would want to
      track plugin session variables changes and these changes would actually
      happen frequently. If this ever becomes an issue, there're much better
      ways to optimise this workload.
      
      Part of MDEV-14984 - regression in connect performance
      894df7ed
    • Sergey Vojtovich's avatar
      Make connect speed great again · 53671a1f
      Sergey Vojtovich authored
      Rather than parsing session_track_system_variables when thread starts, do
      it when first trackable event occurs.
      
      Benchmarked on a 2socket/20core/40threads Broadwell system using sysbench
      connect brencmark @40 threads (with select 1 disabled):
      101379.77 -> 143016.68 CPS, whereas 10.2 is currently at 137766.31 CPS.
      
      Part of MDEV-14984 - regression in connect performance
      53671a1f
    • Sergey Vojtovich's avatar
      Safe session_track_system_variables snapshot · 1b5cf2f7
      Sergey Vojtovich authored
      When enabling system variables tracker, make a copy of
      global_system_variables.session_track_system_variables
      under LOCK_global_system_variables. This protects from concurrent variable
      updates and potential freed memory access, as well as from variable
      reconstruction (which was previously protected by LOCK_plugin).
      
      We can also use this copy as a session variable pointer, so that we don't
      have to allocate memory and reconstruct it every time it is referenced.
      
      For this very reason we don't need buffer_length stuff anymore.
      
      As well as don't we need to take LOCK_plugin early in ::enable().
      Unified ::parse_var_list() to acquire LOCK_plugin unconditionally.
      For no apparent reason it wasn't previously acquired for global
      variable update.
      
      Part of MDEV-14984 - regression in connect performance
      1b5cf2f7
    • Sergey Vojtovich's avatar
      Allocate Session_sysvars_tracker statically · 554ac6f3
      Sergey Vojtovich authored
      One less new/delete per connection.
      
      Removed m_mem_flag since most allocs are thread specific. The only
      exception are allocs performed during initialization.
      
      Removed State_tracker and Session_tracker constructors as they don't make
      sense anymore.
      
      No reason to access session_sysvars_tracker via get_tracker(), so access
      it directly instead.
      
      Part of MDEV-14984 - regression in connect performance
      554ac6f3
    • Sergey Vojtovich's avatar
      Allocate Transaction_state_tracker statically · a7adc2ce
      Sergey Vojtovich authored
      One less new/delete per connection.
      
      Part of MDEV-14984 - regression in connect performance
      a7adc2ce
    • Sergey Vojtovich's avatar
      Static current schema and state change trackers · 47bd06d5
      Sergey Vojtovich authored
      Saves 2 new/delete per connection.
      
      Part of MDEV-14984 - regression in connect performance
      47bd06d5
    • Sergey Vojtovich's avatar
      Allocate orig_list statically · 01e8f3c5
      Sergey Vojtovich authored
      tool_list is a temporary list needed only for SET SESSION
      session_track_system_variables, so allocate it on stack instead.
      
      Sane reinit() calls.
      
      Saves 2 new/delete per connection.
      
      Part of MDEV-14984 - regression in connect performance
      01e8f3c5
    • Sergey Vojtovich's avatar
      Get rid of not implemented SESSION_GTIDS_TRACKER · 55bdd7f7
      Sergey Vojtovich authored
      One less new/delete per connection.
      
      Part of MDEV-14984 - regression in connect performance
      55bdd7f7
    • Sergey Vojtovich's avatar
      Cleanup session tracker API · 2be28a91
      Sergey Vojtovich authored
      - Session_sysvars_tracker::server_init_check() -> sysvartrack_validate_value()
      - Session_sysvars_tracker::check_var_list()    -> sysvartrack_validate_value()
      
      - Session_sysvars_tracker::server_init_process() -> sysvartrack_global_update()
      - sysvartrack_reprint_value()                    -> sysvartrack_global_update()
      
      - sysvartrack_value_len()       -> sysvartrack_session_value_ptr()
      - sysvartrack_value_construct() -> sysvartrack_session_value_ptr()
      
      - sysvartrack_update() -> Session_sysvars_tracker::update()
      - Session_tracker::server_boot_verify() -> session_tracker_init()
      - sysvar_tracker() -> /dev/null
      
      Part of MDEV-14984 - regression in connect performance
      2be28a91
    • Sergey Vojtovich's avatar
      Cleanup session tracker redundancy · 19d5ddcc
      Sergey Vojtovich authored
      - m_enabled is initialised by the base class constructor
      - removed unused schema_track_inited
      - moved Transaction_state_tracker constructor to declaration
      - common enable()
      - removed unused Session_sysvars_tracker::check_str()
      - classes are "private" by default
      - don't even try to compile for embedded
      
      Part of MDEV-14984 - regression in connect performance
      19d5ddcc
    • Sergey Vojtovich's avatar
      Session_sysvars_tracker::vars_list cleanups · 8f594b33
      Sergey Vojtovich authored
      - return proper type
      - removed useless node argument
      - removed useless mem_flag
      - classes are "private" by default
      - simplified iterators
      
      Part of MDEV-14984 - regression in connect performance
      8f594b33
    • Sergey Vojtovich's avatar
      A proper State_tracker::m_changed enacpsulation · 0e91e0c3
      Sergey Vojtovich authored
      Note: preserved original behaviour, where remaining trackers are not reset
      on error from store(). This effectively means subsequent statements will
      start tracking from unclean state.
      
      Part of MDEV-14984 - regression in connect performance
      0e91e0c3
    • Varun Gupta's avatar
      MDEV-18943: Group Concat with limit not working with views · 879878e4
      Varun Gupta authored
      Adjusted the Item_func_group_concat::print function to take into account
      limit if present with GROUP_CONCAT
      879878e4
  4. 02 May, 2019 8 commits
  5. 01 May, 2019 3 commits
    • Anel Husakovic's avatar
      MDEV-17654 Incorrect syntax returned for column with CHECK constraint in the... · 2370eeb0
      Anel Husakovic authored
      MDEV-17654 Incorrect syntax returned for column with CHECK constraint in the "SHOW CREATE TABLE ..." result
      
      Prepend COMMENT before CHECK constraint in SHOW CREATE
      
      Closes #924
      2370eeb0
    • Daniel Black's avatar
      MDEV-15051: signal handler - output information about the core generation · dc8e15db
      Daniel Black authored
      The working directory, resource limits and core pattern will
      aid the user finding a core file in the case of failure.
      
      While the core file size is most relevant however other resource
      limits may give a clue as the the cause of the fatal signal so
      include them also.
      
      As signal handler functions are limited, proc filesystem reads/
      readlink calls are used instead of the more obvious getcwd/getrlimits
      functions which aren't listed as signal safe.
      
      Results in output of the form:
      
      Writing a core file: working directory at /tmp/datadir
      Resource Limits:
      Limit                     Soft Limit           Hard Limit Units
      Max cpu time              unlimited            unlimited seconds
      Max file size             unlimited            unlimited bytes
      Max data size             unlimited            unlimited bytes
      Max stack size            8388608              unlimited bytes
      Max core file size        unlimited            unlimited bytes
      Max resident set          unlimited            unlimited bytes
      Max processes             47194                47194 processes
      Max open files            1024                 4096 files
      Max locked memory         65536                65536 bytes
      Max address space         unlimited            unlimited bytes
      Max file locks            unlimited            unlimited locks
      Max pending signals       47194                47194 signals
      Max msgqueue size         819200               819200 bytes
      Max nice priority         0                    0
      Max realtime priority     0                    0
      Max realtime timeout      unlimited            unlimited            us
      Core pattern: |/usr/libexec/abrt-hook-ccpp %s %c %p %u %g %t %P %I
      
      Segmentation fault (core dumped)
      
      Closes #537
      dc8e15db
    • Sergei Golubchik's avatar
      compilation fixes for VS 2019 · b953bf7e
      Sergei Golubchik authored
      b953bf7e
  6. 30 Apr, 2019 3 commits
  7. 29 Apr, 2019 4 commits
    • Marko Mäkelä's avatar
      Merge 10.2 into 10.3 · 447b8ba1
      Marko Mäkelä authored
      447b8ba1
    • Marko Mäkelä's avatar
      MDEV-18429: Simpler implementation · 810f014c
      Marko Mäkelä authored
      This reverts commit 61f370a3
      and implements a simpler fix that is straightforward to merge to 10.3.
      
      lock_print_info: Renamed from PrintNotStarted. Dump the
      entire contents of trx_sys->mysql_trx_list.
      
      lock_print_info_rw_recovered: Like lock_print_info, but dump
      only recovered transactions in trx_sys->rw_trx_list.
      
      lock_print_info_all_transactions(): Dump both trx_sys->mysql_trx_list
      and trx_sys->rw_trx_list.
      
      TrxLockIterator, TrxListIterator, lock_rec_fetch_page(): Remove.
      This is a partial backport of the 10.3
      commit a447980f
      which removed the race-condition-prone ability of the InnoDB monitor
      to read relevant pages into the buffer pool for some record locks.
      810f014c
    • Marko Mäkelä's avatar
      MDEV-14130 InnoDB messages should not refer to the MySQL 5.7 manual · 092602ac
      Marko Mäkelä authored
      In InnoDB error messages, replace the hyperlink URLs to point
      to the MariaDB knowledge base.
      092602ac
    • Marko Mäkelä's avatar
      Clean up ut_list · 5fb4c0a8
      Marko Mäkelä authored
      ut_list_validate(), ut_list_map(): Add variants with const Functor&
      so that these functions can be called with an rvalue.
      
      Remove wrapper macros, and add #ifdef UNIV_DEBUG around debug-only code.
      5fb4c0a8