1. 04 Jan, 2008 3 commits
  2. 03 Jan, 2008 2 commits
  3. 20 Dec, 2007 7 commits
    • marko's avatar
      branches/zip: Allow a build with UNIV_PAGE_SIZE set to 4096. · 8a882081
      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.
      8a882081
    • marko's avatar
      branches/zip: Fix some things to allow InnoDB to be built with smaller · 2a2a53bd
      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.
      2a2a53bd
    • vasil's avatar
      branches/zip: · ff9ef522
      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)
      ff9ef522
    • vasil's avatar
      branches/zip: · 74ac13f0
      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
      74ac13f0
    • marko's avatar
      branches/zip: trx0sys.h: Add a compile-time check that UNIV_PAGE_SIZE · 81b55378
      marko authored
      is at least 4 kilobytes.
      81b55378
    • marko's avatar
      branches/zip: Allow the uncompressed page size to be settable by setting · cd353a2b
      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.
      cd353a2b
    • marko's avatar
      branches/zip: Add some clarifying comments. · 7696aaf8
      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).
      7696aaf8
  4. 19 Dec, 2007 4 commits
    • marko's avatar
      branches/zip: Minor cleanup. · d35922d1
      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.
      d35922d1
    • marko's avatar
      branches/zip: Fast index creation: Lock the data dictionary only after · e3d19b0f
      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().
      e3d19b0f
    • marko's avatar
      branches/zip: Implement a limit for the size of undo log records. · 463141d1
      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.
      463141d1
    • marko's avatar
      branches/zip: dict0dict.c: Minor cleanup. · b33d8c40
      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[].
      b33d8c40
  5. 18 Dec, 2007 1 commit
    • vasil's avatar
      branches/zip: · 204f29ac
      vasil authored
      Non-functional change:
      Do not include the terminating '\0' in TRX_I_S_LOCK_ID_MAX_LEN.
      204f29ac
  6. 17 Dec, 2007 7 commits
    • marko's avatar
      branches/zip: Fast index creation: Clarify why lock waits may occur in · 6690972b
      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.
      6690972b
    • marko's avatar
    • marko's avatar
      branches/zip: Fast index creation: Remove the ROW_PREBUILT_OBSOLETE nonsense. · d0d92991
      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.
      d0d92991
    • marko's avatar
      branches/zip: lock_rec_insert_check_and_lock(): Use the cached value · da28dd3c
      marko authored
      of thr_get_trx(thr).
      da28dd3c
    • marko's avatar
      branches/zip: innobase_mysql_end_print_arbitrary_thd(): Note that · ca75e779
      marko authored
      kernel_mutex must be released before calling this function.
      
      innobase_mysql_end_print_arbitrary_thd(),
      innobase_mysql_prepare_print_arbitrary_thd(): Assert that the
      kernel_mutex is not being held by the current thread.
      ca75e779
    • vasil's avatar
      branches/zip: · 74ab4b66
      vasil authored
       
      Bugfix: Lock the MySQL mutex LOCK_thread_count before accessing
      trx->mysql_query_str to avoid race conditions where MySQL sets it to
      NULL after we have checked that it is not NULL and before we access it.
       
      Approved by:	Marko
      74ab4b66
    • vasil's avatar
      branches/zip: · 815bb2cc
      vasil authored
      Non-functional change: add "out:" comment for the return value.
      815bb2cc
  7. 16 Dec, 2007 1 commit
    • vasil's avatar
      branches/zip: · 0845bd2d
      vasil authored
       
      Non-functional change:
       
      Move the prototypes of
      innobase_mysql_prepare_print_arbitrary_thd() and
      innobase_mysql_end_print_arbitrary_thd() from lock0lock.c to
      ha_prototypes.h
      
      Suggested by:	Marko
      Approved by:	Marko
      0845bd2d
  8. 13 Dec, 2007 5 commits
    • marko's avatar
      branches/zip: page_zip_decompress(): Implement a proper check if there · 476cadbf
      marko authored
      is an overlap between BLOB pointers and the modification log or the
      zlib stream.
      
      page_zip_decompress_clust_ext(): Remove the improper check.  The
      d_stream->avail_in cannot be decremented here, because we do not know
      at this point if the record is deleted.  No space is reserved for the
      BLOB pointers in deleted records.
      
      page_zip_decompress_clust(): Check for the overlap here, right before
      copying the BLOB pointers.
      
      page_zip_decompress_clust(): Also check that the target column is long
      enough, and return FALSE instead of ut_ad() failure.
      476cadbf
    • vasil's avatar
      branches/zip: · 81ff3ffd
      vasil authored
      Add some clarification to a comment.
      81ff3ffd
    • marko's avatar
      branches/zip: page_zip_decompress_node_ptrs(): Remove the local variable · e525556f
      marko authored
      is_clust, to avoid a warning about unused variable when the definition
      of page_zip_fail() is empty.
      e525556f
    • marko's avatar
      branches/zip: page0zip.c: Add more page_zip_fail() diagnostics to · 189aabfd
      marko authored
      some decompression functions.
      
      page_zip_apply_log_ext(), page_zip_apply_log(): Call page_zip_fail()
      with appropriate diagnostics before returning NULL.
      
      page_zip_decompress_node_ptrs(), page_zip_decompress_sec(),
      page_zip_decompress_clust(): When detecting that the zlib stream
      followed by the modification log overlaps the trailer, do not
      let an assertion fail, but invoke page_zip_fail() and return FALSE.
      Corrupt data should never lead into assertion failures in decompression
      functions.
      189aabfd
    • marko's avatar
      branches/zip: page0zip.c: Define and use the auxiliary macros · d6a5ad85
      marko authored
      ASSERT_ZERO() and ASSERT_ZERO_BLOB() for asserting that certain
      blocks of memory are filled with zero.
      d6a5ad85
  9. 12 Dec, 2007 2 commits
  10. 10 Dec, 2007 3 commits
  11. 07 Dec, 2007 2 commits
    • marko's avatar
      branches/zip: rec_convert_dtuple_to_rec_comp(): Allow externally stored · 132d888b
      marko authored
      columns to be up to REC_MAX_INDEX_COL_LEN + BTR_EXTERN_FIELD_REF_SIZE
      bytes in a debug assertion.  This assertion could fail since r2159 in
      trx_undo_prev_version_build(), because the undo log records for updates
      and deletes would contain longer prefixes of externally stored columns.
      
      The assertion failure was reported by Sunny.
      132d888b
    • marko's avatar
      branches/zip: dict_table_copy_types(): Initialize all fields to the SQL NULL · 5a6cc213
      marko authored
      value.  Document this change in behaviour, and make all callers invoke
      the function right after dtuple_create().
      
      dict_create_sys_fields_tuple(): Add a missing "break" statement to the loop
      that checks if there are any column prefixes in the index.
      
      row_get_prebuilt_insert_row(): Do not set the fields to the SQL NULL value,
      now that dict_table_copy_types() takes care of it.
      5a6cc213
  12. 05 Dec, 2007 3 commits
    • marko's avatar
      branches/zip: When logging updates or deletes in the undo log, store long · 93877450
      marko authored
      enough prefixes of externally stored columns, so that purge will not have
      to dereference any BLOB pointers, which may be invalid.  This will not be
      necessary for logging inserts, because inserts are no-ops in purge, and
      the record will remain locked during transaction rollback.
      
      TODO: in dict_build_table_def_step() or dict_build_index_def_step(),
      prevent the creation of tables with too many columns for which a
      prefix index is defined.  This is because there is a size limit of undo
      log records, and for each prefix-indexed column, the log must store
      REC_MAX_INDEX_COL_LEN + BTR_EXTERN_FIELD_REF_SIZE bytes.
      
      trx_undo_page_report_insert(): Assert that the index is clustered.
      
      trx_undo_page_fetch_ext(): New function, for fetching the BLOB prefix
      in trx_undo_page_report_modify().
      
      trx_undo_page_report_modify(): Write long enough prefixes of the externally
      stored columns to the undo log.
      
      trx_undo_rec_get_partial_row(): Remove the parameter "ext".  Assert that
      the undo log contains long enough prefixes of the externally stored columns.
      
      purge_node_t: Remove the field "ext".
      93877450
    • marko's avatar
      branches/zip: row_build_index_entry(): Add assertions that prevent improper · 9c975c59
      marko authored
      prefix indexes from being built on externally stored columns.
      9c975c59
    • marko's avatar
      branches/zip: btr_cur_pessimistic_update(), btr_cur_pessimistic_delete(): · 89a51cc8
      marko authored
      Use rec_offs_any_extern() as a condition for freeing externally stored
      columns.  This is only a performance optimization.
      89a51cc8