1. 16 Jan, 2008 4 commits
    • marko's avatar
      branches/zip: Add comments about the lock and latch protection of externally · e693d517
      marko authored
      stored columns (BLOBs).
      
      btr_copy_blob_prefix(), btr_copy_zblob_prefix(),
      btr_copy_externally_stored_field_prefix_low(),
      btr_copy_externally_stored_field_prefix(),
      btr_copy_externally_stored_field(),
      btr_rec_copy_externally_stored_field():
      Note that the page containing the clustered index record that points to
      the BLOB must be latched.
      
      btr_copy_zblob_prefix(): Note that there is no latch on the page, and thus
      all accesses to a given page via this function must be covered by the same
      set of locks or latches.
      
      btr_copy_zblob_prefix(): Note that the block acquired by
      buf_page_get_zip() is protected by an exclusive table lock or
      or by a latch on the clustered index record.
      e693d517
    • marko's avatar
      branches/zip: Fetch externally stored columns only when the clustered index · da25388d
      marko authored
      record is protected by a latch or a lock.
      
      dtuple_copy(): New function: Copy a data tuple.
      
      row_upd_replace(): New function: Apply on a row an update vector that
      was built for the clustered index.  Set up a cache of externally stored
      column prefixes if needed.
      
      undo_node_t: Add the fields undo_row, undo_ext.
      
      row_undo_search_clust_to_pcur(): Initialize undo_row and undo_ext.
      
      row_undo_mod_upd_exist_sec(): Instead of fetching prefixes of
      externally stored columns, use the undo_row and undo_ext that were
      initialized in row_undo_search_clust_to_pcur().
      
      upd_node_t: Remove the field n_ext.  Add the fields upd_row and upd_ext.
      
      row_upd_store_row(): Initialize the upd_row and upd_ext fields of upd_node_t.
      
      row_upd_sec_index_entry(), row_upd_clust_rec_by_insert(): Instead of
      fetching prefixes of externally stored columns, use the upd_row
      and upd_ext that were initialized in row_upd_store_row().
      da25388d
    • marko's avatar
      branches/zip: row_undo_ins_parse_undo_rec(): Initialize node->update · 9188a049
      marko authored
      so that it will not be uninitialized when the undo record is being
      processed.
      9188a049
    • vasil's avatar
      branches/zip: · 12c03b43
      vasil authored
       
      Move the column trx_weight near the end in the
      information_schema.innodb_trx table.
      
      Suggested by:	Ken
      Approved by:	Marko
      12c03b43
  2. 15 Jan, 2008 2 commits
  3. 14 Jan, 2008 3 commits
  4. 11 Jan, 2008 3 commits
    • marko's avatar
      branches/zip: Clarify why certain calls of · aaccf926
      marko authored
      btr_rec_copy_externally_stored_field() are safe.
      
      row_merge_copy_blobs(): Note that the table is locked during index creation.
      Therefore, none of its BLOBs can be freed.
      
      row_sel_fetch_columns(): Note that rec must be protected by a page latch.
      Add const qualifier to rec.
      
      row_sel_get_clust_rec(): Note that the clustered index record is protected
      by a page latch that was acquired when the persistent cursor was positioned
      and that the latch will be freed by mini-transaction commit.
      
      row_sel_try_search_shortcut(): Check the delete-mark flag before fetching
      the columns.  Note that the clustered index record is protected
      by a page latch that was acquired when the persistent cursor was positioned
      and that the latch will be freed by mini-transaction commit.
      
      row_sel(), row_search_for_mysql(): Note that the clustered index record
      is protected by a page latch that was acquired when the persistent cursor
      was positioned and that the latch will be freed by mini-transaction commit.
      
      row_sel_field_store_in_mysql_format(): Add const qualifier to data.
      
      row_sel_store_mysql_rec(), row_sel_push_cache_row_for_mysql():
      Add const qualifier to rec.  Note that rec must be protected by a page latch.
      aaccf926
    • marko's avatar
    • marko's avatar
      branches/zip: Add clarifying comments that the clustered index record must · f6f4a52b
      marko authored
      be protected from being deleted while any externally stored columns are
      being fetched.
      
      row_ext_create(),
      row_upd_index_replace_new_col_vals(),
      row_upd_index_replace_new_col_vals_index_pos(),
      row_sel_sec_rec_is_for_blob(),
      row_sel_sec_rec_is_for_clust_rec(): Note that the clustered
      index record must be covered by a lock or a page latch.
      
      row_upd_sec_index_entry(), row_upd_clust_rec_by_insert(),
      row_undo_mod_upd_exist_sec(): Note that these functions probably
      do not obtain an appropriate lock on the clustered index record
      before fetching any externally stored columns.
      f6f4a52b
  5. 10 Jan, 2008 4 commits
    • marko's avatar
      branches/zip: buf_LRU_search_and_free_block(): Do not forbid the release · 0d025934
      marko authored
      of the buffer pool mutex.  Apparently, it is temporarily released also
      in older versions of MySQL/InnoDB for the duration of the
      btr_search_drop_page_hash_index() call [in buf_LRU_free_block()].
      0d025934
    • marko's avatar
      1cbf67cd
    • marko's avatar
      branches/zip: Add instrumentation for prohibiting the release of · 7ac6a343
      marko authored
      the buffer pool mutex.  The instrumentation can be activated by
      defining UNIV_DEBUG or UNIV_BUF_DEBUG at compilation time.
      
      buf_pool_mutex_exit_forbidden: New variable.  When this is nonzero,
      an assertion will fail in buf_pool_mutex_exit().
      
      buf_pool_mutex_exit_forbid(): Macro for declaring that the buffer pool
      mutex must not be released.  Calls may be nested.
      
      buf_pool_mutex_exit_allow(): Macro for declaring that the buffer pool
      mutex may be released.  Calls may be nested.
      
      buf_LRU_search_and_free_block(): Prohibit buf_pool_mutex_exit() in the
      scope of the function.
      
      buf_LRU_free_block(): Prohibit buf_pool_mutex_exit() in buf_buddy_alloc()
      and buf_buddy_free().
      
      buf_LRU_block_remove_hashed_page(): Prohibit buf_pool_mutex_exit()
      in buf_buddy_free().
      7ac6a343
    • marko's avatar
      branches/zip: Implement wrappers for all operations on the buffer pool mutex. · cdb5b464
      marko authored
      buf_pool->mutex: Rename to buf_pool_mutex, so that the wrappers will have
      to be used when changes are merged from other source trees.
      
      buf_pool->zip_mutex: Rename to buf_pool_zip_mutex.
      
      buf_pool_mutex_own(), buf_pool_mutex_enter(), buf_pool_mutex_exit():
      Wrappers for buf_pool_mutex.
      cdb5b464
  6. 09 Jan, 2008 1 commit
    • marko's avatar
      branches/zip: Clean up index->to_be_dropped after a failed DROP INDEX. · 4c8c6f37
      marko authored
      ha_innobase::final_drop_index(): If row_merge_drop_table() fails, clear
      the to_be_dropped flags.  This was the error fixed in this commit; the rest
      is just additional safety.
      
      ha_innobase::final_drop_index(): After dropping the flagged indexes,
      assert that none of the remaining indexes are flagged to_be_dropped.
      
      ha_innobase::prepare_drop_index(): Assert that no index has been flagged
      for deletion.  When checking foreign key constraints, simply traverse the
      list of indexes and check if any of the indexes that were just flagged
      to_be_dropped.  On error, clear the to_be_dropped flags with simple list
      traversal.
      4c8c6f37
  7. 07 Jan, 2008 2 commits
  8. 04 Jan, 2008 4 commits
  9. 03 Jan, 2008 2 commits
  10. 20 Dec, 2007 7 commits
    • marko's avatar
      branches/zip: Allow a build with UNIV_PAGE_SIZE set to 4096. · dfe13a1c
      marko authored
      TRX_RSEG_N_SLOTS: Make the definition dependent of UNIV_PAGE_SIZE.
      
      FSP_EXTENT_SIZE: Define as 1 megabyte, as assumed by the code elsewhere.
      
      fseg_create_general(): Add a debug assertion against page overflow.
      dfe13a1c
    • marko's avatar
      branches/zip: Fix some things to allow InnoDB to be built with smaller · f19db3bb
      marko authored
      UNIV_PAGE_SIZE than the default 16384.
      
      MEM_BLOCK_STANDARD_SIZE: Cap to MEM_MAX_ALLOC_IN_BUF when UNIV_PAGE_SIZE
      is less than 16384.
      
      ha_create_func(): Invoke mem_heap_create_in_btr_search() with
      ut_min(4096, MEM_MAX_ALLOC_IN_BUF) instead of 4096.  The memory
      will be allocated from the buffer pool, and with UNIV_PAGE_SIZE
      defined to 4096, there would not be any space for the mem_heap
      data structure overhead.
      f19db3bb
    • vasil's avatar
      branches/zip: · 85b6ef6d
      vasil authored
      Change the format of TRX_IDs in INFORMATION_SCHEMA tables from DEC to
      HEX.
      
      The current TRX_IDs are hard to remember and track down: 426355, 428466,
      428566, etc.
      
      In HEX:
      * there are less "digits", the strings are shorter;
      * since there are 16 instead of 10 "digits", the chance of having
        repeating ones are smaller.
      
      The above look like 68173, 689B2, 68A16 in HEX.
      
      Discussed with:	Ken
      Approved by:	Heikki (via IM)
      85b6ef6d
    • vasil's avatar
      branches/zip: · 1ae59d9e
      vasil authored
      Change the output format of transaction ids from 2 32bit numbers separated
      by space to a single hex number.
      
      Suggested by:	Heikki
      Approved by:	Heikki
      1ae59d9e
    • marko's avatar
      branches/zip: trx0sys.h: Add a compile-time check that UNIV_PAGE_SIZE · 15cc2608
      marko authored
      is at least 4 kilobytes.
      15cc2608
    • marko's avatar
      branches/zip: Allow the uncompressed page size to be settable by setting · 5c7fb887
      marko authored
      just one parameter: UNIV_PAGE_SIZE_SHIFT.
      
      UNIV_PAGE_SIZE, BUF_BUDDY_SIZES: Define in terms of UNIV_PAGE_SIZE_SHIFT.
      
      BUF_BUDDY_LOW_SHIFT: New macro, to simplify the definition of BUF_BUDDY_LOW
      and BUF_BUDDY_SIZES.
      
      PAGE_ZIP_DIR_SLOT_MASK: Relax the compile-time check.  This bitmask must be
      one less than a power of two, and at least UNIV_PAGE_SIZE - 1.
      5c7fb887
    • marko's avatar
      branches/zip: Add some clarifying comments. · 8c852371
      marko authored
      btr_copy_blob_prefix(), btr_copy_externally_stored_field_prefix_low():
      Document the return value as "number of bytes written", not "bytes written".
      
      trx_undo_page_fetch_ext(): Explain the assertion ut_a(ext_len).
      
      row_build_index_entry(): Explain the assertion ut_a(!ext).
      8c852371
  11. 19 Dec, 2007 4 commits
    • marko's avatar
      branches/zip: Minor cleanup. · 2c4aaa9e
      marko authored
      row_create_index_graph_for_mysql(): Move from row0mysql.c to row0merge.c
      and rename to row_merge_create_index_graph().  Also change the function
      comment to say that the function will create and execute the query graph
      for creating the index.
      
      row_merge_create_index(): Remove redundant assignment to trx->error_state.
      2c4aaa9e
    • marko's avatar
      branches/zip: Fast index creation: Lock the data dictionary only after · acf84dfb
      marko authored
      acquiring the table lock.  The data dictionary should not be locked for
      long periods.  Before this change, in the worst case, the dictionary
      would be locked until the expiration of innodb_lock_wait_timeout.
      
      Virtually, transaction-level locks (locks on database objects, such
      as records and tables) have a latching order level of SYNC_USER_TRX_LOCK,
      which is above any InnoDB rw-locks or mutexes.  However, the latching
      order of SYNC_USER_TRX_LOCK is never checked, not even by UNIV_SYNC_DEBUG.
      
      ha_innobase::add_index(), ha_innobase::final_drop_index(): Invoke
      row_mysql_lock_data_dictionary(trx) only after row_merge_lock_table().
      acf84dfb
    • marko's avatar
      branches/zip: Implement a limit for the size of undo log records. · adbb92be
      marko authored
      innodb-index.test: Add a test with a large number of externally stored
      columns.  Check that there may not be prefix indexes on too many columns.
      
      dict_index_too_big_for_undo(): New function: Check if the undo log may
      overflow.
      
      dict_index_add_to_cache(): Return DB_SUCCESS or DB_TOO_BIG_RECORD.
      Postpone the creation and linking of some data structures, so that
      when dict_index_too_big_for_undo() holds, it will be easier to clean up.
      Check the return status in all callers.
      adbb92be
    • marko's avatar
      branches/zip: dict0dict.c: Minor cleanup. · b99fbdf2
      marko authored
      dict_index_copy(): Remove the prototype, because this static function
      will be defined before its first use.  Add const qualifier to "table".
      
      dict_index_build_internal_clust(), dict_index_build_internal_non_clust():
      Add const qualifier to "table".  Correct the comment about setting indexed[].
      b99fbdf2
  12. 18 Dec, 2007 1 commit
    • vasil's avatar
      branches/zip: · 97025c3a
      vasil authored
      Non-functional change:
      Do not include the terminating '\0' in TRX_I_S_LOCK_ID_MAX_LEN.
      97025c3a
  13. 17 Dec, 2007 3 commits
    • marko's avatar
      branches/zip: Fast index creation: Clarify why lock waits may occur in · 807c2807
      marko authored
      row_merge_lock_table().
      
      ha_innobase::final_drop_index(): Set the dictionary operation mode to
      TRX_DICT_OP_INDEX_MAY_WAIT for the duration of the row_merge_lock_table()
      call.
      807c2807
    • marko's avatar
    • marko's avatar
      branches/zip: Fast index creation: Remove the ROW_PREBUILT_OBSOLETE nonsense. · 71a4f691
      marko authored
      Active transactions must not switch table or index definitions on the fly,
      for several reasons, including the following:
      
       * copied indexes do not carry any history or locking information;
         that is, rollbacks, read views, and record locking would be broken
      
       * huge potential for race conditions, inconsistent reads and writes,
         loss of data, and corruption
      
      Instead of trying to track down if the table was changed during a transaction,
      acquire appropriate locks that protect the creation and dropping of indexes.
      
      innodb-index.test: Test the locking of CREATE INDEX and DROP INDEX.  Test
      that consistent reads work across dropped indexes.
      
      lock_rec_insert_check_and_lock(): Relax the lock_table_has() assertion.
      When inserting a record into an index, the table must be at least IX-locked.
      However, when an index is being created, an IS-lock on the table is
      sufficient.
      
      row_merge_lock_table(): Add the parameter enum lock_mode mode, which must
      be LOCK_X or LOCK_S.
      
      row_merge_drop_table(): Assert that n_mysql_handles_opened == 0.
      Unconditionally drop the table.
      
      ha_innobase::add_index(): Acquire an X or S lock on the table, as appropriate.
      After acquiring an X lock, assert that n_mysql_handles_opened == 1.
      Remove the comments about dropping tables in the background.
      
      ha_innobase::final_drop_index(): Acquire an X lock on the table.
      
      dict_table_t: Remove version_number, to_be_dropped, and prebuilts.
      ins_node_t: Remove table_version_number.
      
      enum lock_mode: Move the definition from lock0lock.h to lock0types.h.
      
      ROW_PREBUILT_OBSOLETE, row_update_prebuilt(), row_prebuilt_table_obsolete():
      Remove.
      
      row_prebuilt_t: Remove the declaration from row0types.h.
      
      row_drop_table_for_mysql_no_commit(): Always print a warning if a table
      was added to the background drop queue.
      71a4f691