1. 26 Jun, 2017 1 commit
  2. 23 Jun, 2017 3 commits
    • Marko Mäkelä's avatar
      Prevent interleaved error log output on InnoDB startup · 0d69d313
      Marko Mäkelä authored
      buf_flush_page_cleaner_coordinator(): Signal the thread creator
      that the error log output regarding setpriority() has been issued.
      
      innobase_start_or_create_for_mysql(): Wait for
      buf_flush_page_cleaner_coordinator() to completely start up.
      
      This prevents sporadic failures of tests that search the server
      error log for InnoDB redo log recovery messages.
      0d69d313
    • Marko Mäkelä's avatar
      Do allow writes for innodb_force_recovery=2 or 3 · 0288fa61
      Marko Mäkelä authored
      While the primary purpose of innodb_force_recovery is to allow
      data to be rescued from an InnoDB instance that would crash due
      to some data corruption, the settings 1, 2, or 3 are relatively
      safe to use and there is no need to prevent write transactions
      in these modes.
      
      The setting innodb_force_recovery=4 and above can cause database
      corruption. For those modes, we already set the flag
      high_level_read_only to disable modifications, except DROP TABLE.
      
      MODIFICATIONS_NOT_ALLOWED_MSG_FORCE_RECOVERY: Remove. There is no
      need to spam the error log for each refused DML operation. It suffices
      to return an error to the client. There will be messages at startup
      if innodb_read_only or innodb_force_recovery are preventing writes.
      0288fa61
    • Igor Babaev's avatar
      Fixed the bug mdev-12845. · 9f362219
      Igor Babaev authored
      This patch fills in a serious flaw in the
      code that supports condition pushdown into
      materialized views / derived tables.
      
      If a predicate happened to contain a reference
      to a mergeable view / derived table and it does
      not depended directly on the target materialized
      view / derived table then the predicate was not
      considered as a subject to pusdown to this view
      / derived table.
      9f362219
  3. 22 Jun, 2017 7 commits
  4. 21 Jun, 2017 1 commit
  5. 20 Jun, 2017 1 commit
    • Vicențiu Ciorbaru's avatar
      MDEV-13068 Crash in Item::split_sum_func2 with INSERT SELECT using window functions · 0992be92
      Vicențiu Ciorbaru authored
      When running setup fields during the final step of insert using select
      the final setup_fields does not have any sum functions. Our current
      condition for calling split_sum_func however would attempt to use an empty
      NULL sum_func_list, if the item contained a window function.
      
      The solution is to not perform another split_sum_func for the item
      containing a window function if we do not actually have a sum_func_list.
      0992be92
  6. 19 Jun, 2017 6 commits
  7. 16 Jun, 2017 5 commits
    • Igor Babaev's avatar
      Fixed the bug mdev-13064. · cf4a6abe
      Igor Babaev authored
      This is another attempt to fix the bug mdev-12992.
      This patch introduces st_select_lex::context_analysis_place for
      the place in SELECT where context analysis is currently performed.
      It's similar to st_select_lex::parsing_place, but it is used at
      the preparation stage.
      cf4a6abe
    • Oleksandr Byelkin's avatar
      result fix · f73507e6
      Oleksandr Byelkin authored
      f73507e6
    • Sergey Vojtovich's avatar
      MDEV-12620 - set lock_wait_timeout = 1;flush tables with read lock; · 056bab08
      Sergey Vojtovich authored
                   lock not released after timeout
      
      Release GRL if FLUSH TABLES phase failed.
      056bab08
    • Marko Mäkelä's avatar
      Remove trx_t::has_search_latch and simplify debug code · 50faeda4
      Marko Mäkelä authored
      When the btr_search_latch was split into an array of latches
      in MySQL 5.7.8 as part of the Oracle Bug#20985298 fix, the "caching"
      of the latch across storage engine API calls was removed, and
      the field trx->has_search_latch would only be set during a short
      time frame in the execution of row_search_mvcc(), which was
      formerly called row_search_for_mysql().
      
      This means that the column
      INFORMATION_SCHEMA.INNODB_TRX.TRX_ADAPTIVE_HASH_LATCHED will always
      report 0. That column cannot be removed in MariaDB 10.2, but it
      can be removed in future releases.
      
      trx_t::has_search_latch: Remove.
      
      trx_assert_no_search_latch(): Remove.
      
      row_sel_try_search_shortcut_for_mysql(): Remove a redundant condition
      on trx->has_search_latch (it was always true).
      
      sync_check_iterate(): Make the parameter const.
      
      sync_check_functor_t: Make the operator() const, and remove result()
      and the virtual destructor. There is no need to have mutable state
      in the functors.
      
      sync_checker<bool>: Replaces dict_sync_check and btrsea_sync_check.
      
      sync_check: Replaces btrsea_sync_check.
      
      dict_sync_check: Instantiated from sync_checker.
      
      sync_allowed_latches: Use std::find() directly on the array.
      Remove the std::vector.
      
      TrxInInnoDB::enter(), TrxInInnoDB::exit(): Remove obviously redundant
      debug assertions on trx->in_depth, and use equality comparison against 0
      because it could be more efficient on some architectures.
      50faeda4
    • Marko Mäkelä's avatar
      Remove the unnecessary method handlerton::release_temporary_latches() · e5980bf1
      Marko Mäkelä authored
      The sole purpose of handlerton::release_temporary_latches and its wrapper
      function was to release the InnoDB adaptive hash index latch
      (btr_search_latch).
      
      When the btr_search_latch was split into an array of latches
      in MySQL 5.7.8 as part of the Oracle Bug#20985298 fix, the "caching"
      of the latch across storage engine API calls was removed. As part of that,
      the function trx_search_latch_release_if_reserved() was changed to an
      assertion and the function trx_reserve_search_latch_if_not_reserved()
      was removed, and handlerton::release_temporary_latches() practically
      became a no-op.
      
      Note: MDEV-12121 replaced the function
      trx_search_latch_release_if_reserved()
      with the more appropriately named macro trx_assert_no_search_latch().
      e5980bf1
  8. 15 Jun, 2017 5 commits
    • Marko Mäkelä's avatar
      Follow-up to MDEV-12873: Refactor SYS_TABLES.TYPE validation · 6b71b3e3
      Marko Mäkelä authored
      dict_sys_tables_type_to_tf(): Change the parameter n_cols to not_redundant.
      
      dict_tf_is_valid_not_redundant(): Refactored from dict_tf_is_valid().
      
      dict_sys_tables_type_valid(): Replaces dict_sys_tables_type_validate().
      Use the common function dict_tf_is_valid_not_redundant(), which validates
      PAGE_COMPRESSION_LEVEL more strictly.
      
      DICT_TF_GET_UNUSED(flags): Remove.
      6b71b3e3
    • Marko Mäkelä's avatar
      Merge 10.1 into 10.2 · 615b1f41
      Marko Mäkelä authored
      innodb.table_flags: Adjust the test case. Due to the MDEV-12873 fix
      in 10.2, the corrupted flags for table test.td would be converted,
      and a tablespace flag mismatch will occur when trying to open the file.
      615b1f41
    • Marko Mäkelä's avatar
      MDEV-12873 InnoDB SYS_TABLES.TYPE incompatibility for PAGE_COMPRESSED=YES in... · 72378a25
      Marko Mäkelä authored
      MDEV-12873 InnoDB SYS_TABLES.TYPE incompatibility for PAGE_COMPRESSED=YES in MariaDB 10.2.2 to 10.2.6
      
      Remove the SHARED_SPACE flag that was erroneously introduced in
      MariaDB 10.2.2, and shift the SYS_TABLES.TYPE flags back to where
      they were before MariaDB 10.2.2. While doing this, ensure that
      tables created with affected MariaDB versions can be loaded,
      and also ensure that tables created with MySQL 5.7 using the
      TABLESPACE attribute cannot be loaded.
      
      MariaDB 10.2.2 picked the SHARED_SPACE flag from MySQL 5.7,
      shifting the MariaDB 10.1 flags PAGE_COMPRESSION, PAGE_COMPRESSION_LEVEL,
      ATOMIC_WRITES by one bit. The SHARED_SPACE flag would always
      be written as 0 by MariaDB, because MariaDB does not support
      CREATE TABLESPACE or CREATE TABLE...TABLESPACE for InnoDB.
      
      So, instead of the bits AALLLLCxxxxxxx we would have
      AALLLLC0xxxxxxx if the table was created with MariaDB 10.2.2
      to 10.2.6. (AA=ATOMIC_WRITES, LLLL=PAGE_COMPRESSION_LEVEL,
      C=PAGE_COMPRESSED, xxxxxxx=7 bits that were not moved.)
      
      PAGE_COMPRESSED=NO implies LLLLC=00000. That is not a problem.
      
      If someone created a table in MariaDB 10.2.2 or 10.2.3 with
      the attribute ATOMIC_WRITES=OFF (value 2; AA=10) and without
      PAGE_COMPRESSED=YES or PAGE_COMPRESSION_LEVEL, the table should be
      rejected. We ignore this problem, because it should be unlikely
      for anyone to specify ATOMIC_WRITES=OFF, and because 10.2.2 and
      10.2.2 were not mature releases. The value ATOMIC_WRITES=ON (1)
      would be interpreted as ATOMIC_WRITES=OFF, but starting with
      MariaDB 10.2.4 the ATOMIC_WRITES attribute is ignored.
      
      PAGE_COMPRESSED=YES implies that PAGE_COMPRESSION_LEVEL be between
      1 and 9 and that ROW_FORMAT be COMPACT or DYNAMIC. Thus, the affected
      wrong bit pattern in SYS_TABLES.TYPE is of the form AALLLL10DB00001
      where D signals the presence of a DATA DIRECTORY attribute and B is 1
      for ROW_FORMAT=DYNAMIC and 0 for ROW_FORMAT=COMPACT. We must interpret
      this bit pattern as AALLLL1DB00001 (discarding the extraneous 0 bit).
      
      dict_sys_tables_rec_read(): Adjust the affected bit pattern when
      reading the SYS_TABLES.TYPE column. In case of invalid flags,
      report both SYS_TABLES.TYPE (after possible adjustment) and
      SYS_TABLES.MIX_LEN.
      
      dict_load_table_one(): Replace an unreachable condition on
      !dict_tf2_is_valid() with a debug assertion. The flags will already
      have been validated by dict_sys_tables_rec_read(); if that validation
      fails, dict_load_table_low() will have failed.
      
      fil_ibd_create(): Shorten an error message about a file pre-existing.
      
      Datafile::validate_to_dd(): Clarify an error message about tablespace
      flags mismatch.
      
      ha_innobase::open(): Remove an unnecessary warning message.
      
      dict_tf_is_valid(): Simplify and stricten the logic. Validate the
      values of PAGE_COMPRESSION. Remove error log output; let the callers
      handle that.
      
      DICT_TF_BITS: Remove ATOMIC_WRITES, PAGE_ENCRYPTION, PAGE_ENCRYPTION_KEY.
      The ATOMIC_WRITES is ignored once the SYS_TABLES.TYPE has been validated;
      there is no need to store it in dict_table_t::flags. The PAGE_ENCRYPTION
      and PAGE_ENCRYPTION_KEY are unused since MariaDB 10.1.4 (the GA release
      was 10.1.8).
      
      DICT_TF_BIT_MASK: Remove (unused).
      
      FSP_FLAGS_MEM_ATOMIC_WRITES: Remove (the flags are never read).
      
      row_import_read_v1(): Display an error if dict_tf_is_valid() fails.
      72378a25
    • Marko Mäkelä's avatar
      Remove some fields from dict_table_t · 58f87a41
      Marko Mäkelä authored
      dict_table_t::thd: Remove. This was only used by btr_root_block_get()
      for reporting decryption failures, and it was only assigned by
      ha_innobase::open(), and never cleared. This could mean that if a
      connection is closed, the pointer would become stale, and the server
      could crash while trying to report the error. It could also mean
      that an error is being reported to the wrong client. It is better
      to use current_thd in this case, even though it could mean that if
      the code is invoked from an InnoDB background operation, there would
      be no connection to which to send the error message.
      
      Remove dict_table_t::crypt_data and dict_table_t::page_0_read.
      These fields were never read.
      
      fil_open_single_table_tablespace(): Remove the parameter "table".
      58f87a41
    • Marko Mäkelä's avatar
      88b96181
  9. 14 Jun, 2017 3 commits
    • Oleksandr Byelkin's avatar
      227bfe44
    • Oleksandr Byelkin's avatar
      MDEV-12471: BULK Command · 91ae1258
      Oleksandr Byelkin authored
      BULK execution moved to a new command.
      91ae1258
    • Marko Mäkelä's avatar
      MDEV-13084 MariaDB 10.2 crashes on corrupted SYS_TABLES.MIX_LEN field · e813fe86
      Marko Mäkelä authored
      innodb.row_format_redundant: Really corrupt the SYS_TABLES.MIX_LEN,
      and do not use any debug instrumentation. For tables created in the
      system tablespace, the contents of the column will be ignored.
      Only the table t1 will refuse to load.
      
      dict_load_table_one(): Remove the DBUG_EXECUTE_IF instrumentation.
      Omit a redundant error message "incorrect flags in SYS_TABLES".
      
      dict_sys_tables_rec_read(): Partially revert the Oracle Bug#21644827
      fix, and always report errors by the return value.
      
      fts_create_in_mem_aux_table(): Do not rely on dict_table_t::flags2,
      but instead evaluate the tablespace ID.
      
      DICT_TF2_BITS: Reduce to the correct value of 7. The two extra
      high-order bits were specific to MySQL 5.7.
      e813fe86
  10. 13 Jun, 2017 3 commits
  11. 12 Jun, 2017 5 commits