1. 27 Sep, 2007 11 commits
  2. 26 Sep, 2007 8 commits
    • marko's avatar
      branches/zip: Document the minimum size of row_merge_block_t as UNIV_PAGE_SIZE. · f8dffe6a
      marko authored
      row_merge_buf_add(): Add ut_ad(data_size < sizeof(row_merge_block_t))
      and document why it may fail if sizeof row_merge_block_t < UNIV_PAGE_SIZE.
      f8dffe6a
    • marko's avatar
      branches/zip: row_merge_build_indexes(): Call innobase_rec_reset() only once. · 9d234dd0
      marko authored
      row_merge_dup_report(): Do not call innobase_rec_reset().
      9d234dd0
    • marko's avatar
      branches/zip: Clean up some non-inlined functions. · 6b3579a8
      marko authored
      dtuple_create_for_mysql(), dtuple_free_for_mysql(): Remove.
      
      ha_innobase::records_in_range(): Use mem_heap_create(), mem_heap_free(),
      and dtuple_create() instead of the removed functions above.  Since r1587,
      InnoDB C++ functions can invoke inlined C functions.
      6b3579a8
    • marko's avatar
      branches/zip: Implement the reporting of duplicate key values to MySQL. · 03a7cb22
      marko authored
      innobase_rec_to_mysql(): New function, for converting an InnoDB clustered
      index record to MySQL table->record[0].  TODO: convert integer fields.
      Currently, integer fields are in big-endian byte order instead of
      host byte order, and signed integer fields are offset by 0x80000000.
      
      innobase_rec_reset(): New function, for resetting table->record[0].
      
      row_merge_build_indexes(): Add the parameter TABLE* table (the MySQL table
      handle) for reporting duplicate key values.
      
      dtuple_from_fields(): New function, to convert an array of dfield_t* to
      dtuple_t.
      
      dtuple_get_n_ext(): New function, to compute the number of externally stored
      fields.
      
      row_merge_dup_t: Structure for counting and reporting duplicate records.
      
      row_merge_dup_report(): Function for counting and reporting duplicate records.
      
      row_merge_tuple_cmp(), row_merge_tuple_sort(): Replace the ulint* n_dup
      parameter with row_merge_dup_t* dup.
      
      row_merge_buf_sort(): Add the parameter row_merge_dup_t* dup, which is
      NULL when sorting a non-unique index.
      
      row_merge_buf_write(), row_merge_heap_create(), row_merge_read_rec(),
      row_merge_cmp(), row_merge_read_clustered_index(), row_merge_blocks(),
      row_merge(), row_merge_sort(): Add const qualifiers.
      
      row_merge_read_clustered_index(): Use a common error handling branch err_exit.
      Invoke row_merge_buf_sort() differently on unique indexes.
      
      row_merge_blocks(): note TODO: We could invoke innobase_rec_to_mysql()
      to report duplicate key values when creating a clustered index.
      03a7cb22
    • marko's avatar
      branches/zip: Minor cleanup. · d0631476
      marko authored
      dict_find_index_by_max_id(): Rename this static function to its
      only caller, dict_table_get_index_by_max_id().
      
      dict_table_get_index_by_max_id(): Copy the function comment from
      dict_find_index_by_max_id().
      d0631476
    • marko's avatar
      branches/zip: Add some const qualifiers to input parameters. · ff9f1a14
      marko authored
      rec_get_converted_size_comp(), rec_convert_dtuple_to_rec_comp(),
      rec_convert_dtuple_to_rec_new(), rec_convert_dtuple_to_rec(): Add a
      const qualifier to dict_index_t*.
      
      row_search_on_row_ref(): Add const qualifiers to the dict_table_t*
      and dtuple_t* parameters.  Note that pcur is an "out" parameter
      and mtr is "in/out".
      ff9f1a14
    • marko's avatar
      branches/zip: Minor cleanup. · 1db629ed
      marko authored
      row_build_row_ref_fast(): Note that "ref" is an in/out parameter.
      
      row_build_row_ref_from_row(): Add const qualifiers to all "in" parameters.
      1db629ed
    • marko's avatar
      branches/zip: Minor cleanup. · 6e0f43ae
      marko authored
      dtuple_create(): Simplify a pointer expression.  Flag the fields uninitialized
      after initializing them in the debug version.
      
      dtuple_t: Only declare magic_n if UNIV_DEBUG is defined.  The field is
      not assigned to nor tested unless UNIV_DEBUG is defined.
      6e0f43ae
  3. 25 Sep, 2007 1 commit
  4. 24 Sep, 2007 6 commits
    • marko's avatar
      branches/zip: Add some const qualifiers or in/out comments to the · 5c9122e7
      marko authored
      dict_index_t* and dict_table_t* parameters of some functions.
      5c9122e7
    • vasil's avatar
      branches/zip: · a4cc4700
      vasil authored
      Copy any data (currently table name and table index) that may be
      destroyed after releasing the kernel mutex into internal cache's
      storage.
      
      This is done in efficient manner using ha_storage type and a given
      string is copied only once into the cache's storage. Later additions of
      the same string use the already stored string, thus allocating memory
      only once per unique string.
      
      Approved by:	Marko
      a4cc4700
    • vasil's avatar
      branches/zip: · 48ed48eb
      vasil authored
      Add a type that stores chunks of data in its own storage and avoids
      duplicates. Supported methods:
      
      ha_storage_create()
      Allocates new storage object.
      
      ha_storage_put()
      Copies a given data chunk into the storage and returns pointer to the
      copy. If the data chunk is already present, a pointer to the existing
      object is returned and the given data chunk is not copied.
      
      ha_storage_empty()
      Clears (empties) the storage from all data chunks that are stored in it.
      
      ha_storage_free()
      Destroys a storage object. Opposite to ha_storage_create().
      
      Approved by:	Marko
      48ed48eb
    • marko's avatar
      branches/zip: Add const qualifiers to the dict_index_t* parameters of · dd1f1c3f
      marko authored
      rec_get_n_fields(), rec_offs_validate(), and rec_offs_make_valid().
      dd1f1c3f
    • marko's avatar
      branches/zip: Restore the size of row_merge_block_t to 1048576 bytes. · e82202fc
      marko authored
      This was inadvertently reduced to 16384 bytes in r1861.  For testing,
      this can be set as low as UNIV_PAGE_SIZE.
      e82202fc
    • marko's avatar
      branches/zip: Correct the bug mentioned in r1872. · 0920bb66
      marko authored
      row_merge(): Add the assertion ut_ad(half > 0).
      
      row_merge_sort(): Compute the half of the merge file correctly.  The
      previous implementation used truncating division, which may result in
      loss of records when the file size in blocks is not a power of 2.
      0920bb66
  5. 22 Sep, 2007 4 commits
    • vasil's avatar
      branches/zip: · 757ca350
      vasil authored
      Non-functional: put the code that clears the IS cache into a separate
      function.
      757ca350
    • vasil's avatar
      branches/zip: · 9d96ad01
      vasil authored
      Cosmetic: initialize the members of the cache in the same order as
      they are defined in the structure.
      9d96ad01
    • vasil's avatar
      branches/zip: · 0b7a91c4
      vasil authored
      Make comment more clear (hopefully).
      0b7a91c4
    • vasil's avatar
      branches/zip: · b20fccc2
      vasil authored
      Use the newly introduced mem_alloc2() to use the memory that has been
      allocated in addition to the requested memory. This is done in order to
      avoid wasting memory.
      
      Do not calculate the sizes and offsets of the chunks in advance in
      table_cache_init() because it is unknown how much bytes will actually
      be allocated by mem_alloc2(). Rather calculate these on the run: after
      each chunk is allocated set its size and the offset of the next chunk.
      
      Similar patch approved by:	Marko
      b20fccc2
  6. 21 Sep, 2007 7 commits
    • marko's avatar
      branches/zip: row_merge_read_clustered_index(): After writing out the · e4ed7666
      marko authored
      merge buffer, write the next record to the beginning of the emptied buffer.
      This fixes one of the bugs mentioned in r1872.
      e4ed7666
    • marko's avatar
      branches/zip: Fix a bug in the merge sort in fast index creation. · fabcf638
      marko authored
      Some bug still remains, because innodb-index.test will lose some
      records from the clustered index after add primary key (a,b(255),c(255))
      when row_merge_block_t is reduced to 8192 bytes.
      
      row_merge(): Add the parameter "half".  Add some Valgrind instrumentation.
      Note that either stream can end before the other one.
      
      row_merge_sort(): Calculate "half" for row_merge().
      fabcf638
    • marko's avatar
      branches/zip: Reduce internal memory fragmentation. · 5c749fa3
      marko authored
      mem_alloc2(): New macro.  This is a variant of mem_alloc() that
      returns the allocated size, which is equal to or greater than
      the requested size.
      
      mem_alloc_func(): Add the output parameter *size for the allocated size.
      When it is set, adjust the parameter passed to mem_heap_alloc().
      
      rec_copy_prefix_to_buf_old(), rec_copy_prefix_to_buf(): Use mem_alloc2()
      instead of mem_alloc().
      5c749fa3
    • marko's avatar
      branches/zip: Add more details to the debug printout enabled by · 5cd8a96f
      marko authored
      row_merge_print_read and row_merge_print_write.
      5cd8a96f
    • marko's avatar
      branches/zip: When creating a memory heap, set the block size to what · 638d5b36
      marko authored
      was actually obtained from the buddy allocator.  This should avoid some
      internal memory fragmentation in mem_heap_create() and mem_heap_alloc().
      
      mem_area_alloc(): Change the in parameter size to an in/out parameter.
      Adjust the size based on what was obtained from pool->free_list[].
      
      mem_heap_create_block(): Adjust block->len to what was obtained from
      mem_area_alloc().
      638d5b36
    • marko's avatar
      branches/zip: innodb-index.test: Add yet another CHECK TABLE and add the · bb97dfee
      marko authored
      column d to two SELECT FROM t1.
      bb97dfee
    • marko's avatar
      branches/zip: Add diagnostic printout. · a44c5905
      marko authored
      rec_print_comp(): New function, sliced from rec_print_new().
      
      rec_print_old(), rec_print_comp(): Print the untruncated length of the column.
      
      row_merge_print_read, row_merge_print_write, row_merge_print_cmp:
      New flags, to enable debug printout in UNIV_DEBUG builds.
      
      row_merge_tuple_print(): New function for UNIV_DEBUG builds.
      
      row_merge_read_rec(): Obey row_merge_print_read.
      
      row_merge_buf_write(), row_merge_write_rec_low(),
      row_merge_write_eof(): Obey row_merge_print_write.
      
      row_merge_cmp(): Obey row_merge_print_cmp.
      a44c5905
  7. 20 Sep, 2007 3 commits