1. 18 Feb, 2008 6 commits
    • marko's avatar
      branches/zip: trx_undo_prev_version_build(): Remove the unnecessary · b48d0c64
      marko authored
      and incorrect "BLOB bug fix" that was suggested by Heikki.  Explain in
      a comment why no such fix is needed.
      b48d0c64
    • marko's avatar
      branches/zip: Add a UNIV_INTERN qualifier to every global function declaration · bddb5a0c
      marko authored
      in *.h files, so that the function signatures in the *.h and *.c files fully
      match each other.
      
      ut_dulint_sort(): Add a UNIV_INTERN qualifier also to the function definition.
      bddb5a0c
    • marko's avatar
      branches/zip: Minor fixes. · 99660234
      marko authored
      lock_rec_restore_from_page_infimum(): Correct the comment of the parameter rec.
      
      lock_sec_rec_read_check_and_lock(): Add the debug assertion
      ut_ad(mode == LOCK_X || mode == LOCK_S).
      99660234
    • marko's avatar
      branches/zip: Minor fixes. · da54bec7
      marko authored
      buf_LRU_old_adjust_len(): Replace a constant ut_ad() with a preprocessor check.
      
      buf_LRU_free_block(): Remove the check for the unlikely case
      buf_pool->LRU_old == prev_b in order to simplify the function.
      The check was implemented as part of r2306.
      da54bec7
    • marko's avatar
      branches/zip: trx_undo_prev_version_build(): Remove the fix that was · 6aa400be
      marko authored
      suggested by Heikki, because it breaks row_vers_impl_x_locked_off_kernel();
      see Mantis issue #10.
      
      However, now that Heikki's fix has been removed, the code may break elsewhere
      when it tries to dereference half-freed or completely freed externally
      stored columns.
      6aa400be
    • marko's avatar
      branches/zip: Minor improvements. · 7a41306b
      marko authored
      lock_update_delete(): Add the debug assertion ut_ad(page == page_align(rec)).
      
      lock_rec_insert_check_and_lock(): Determine next_rec_heap_no before
      acquiring the kernel mutex.  Require the table to be at least S-locked
      in fast index creation.
      7a41306b
  2. 16 Feb, 2008 1 commit
  3. 15 Feb, 2008 5 commits
  4. 13 Feb, 2008 1 commit
    • marko's avatar
      branches/zip: Minor cleanup of fast index creation diagnostics. · 388af7a7
      marko authored
      innobase_check_index_keys(): Remove unused parameters.  Use
      sql_print_error() for error message output.
      
      ha_innobase::add_index(): When row_merge_rename_tables() fails, do not
      allow row_merge_drop_table() to alter the error code returned to MySQL.
      388af7a7
  5. 12 Feb, 2008 1 commit
    • marko's avatar
      branches/zip: Pass -prefer-non-pic only on IA-32 (x86) when building · 9194c3a9
      marko authored
      the dynamic InnoDB plugin, ha_innodb.so.  This libtool option cannot
      be used on every target platform.  For instance, the AMD64 (x86_64) ABI
      mandates position-independent code (PIC) in dynamic shared objects.
      
      Makefile.am: Use $(INNODB_CFLAGS) and $(INNODB_DYNAMIC_CFLAGS) in
      target-specific CFLAGS and CXXFLAGS variable definitions.
      
      plug.in: Instead of appending to the MySQL-wide CFLAGS, introduce
      the variables INNODB_CFLAGS and INNODB_DYNAMIC_CFLAGS.  Depending on
      $target_cpu, append -prefer-non-pic to INNODB_DYNAMIC_CFLAGS.
      9194c3a9
  6. 11 Feb, 2008 1 commit
    • marko's avatar
      branches/zip: Makefile.am: Disable the generation of · 775fd54d
      marko authored
      position-independent code in the compilation of ha_innodb.so.
      
      ha_innodb_la_CXXFLAGS, ha_innodb_la_CFLAGS: Add -prefer-non-pic.
      This flag is interpreted by the ../../libtool script that acts
      as a front-end to the compiler and linker.
      775fd54d
  7. 08 Feb, 2008 6 commits
  8. 07 Feb, 2008 3 commits
    • inaam's avatar
      branches/zip: · 044e405d
      inaam authored
      srv_log_file_size is misspelled as srv_log_filen_size in r2276
      
      reviewed by: non-functional change.
      044e405d
    • marko's avatar
      branches/zip: make_flex.sh, lexyy.c: Remove some GCC warnings about unused · c5c5fdc5
      marko authored
      static symbol definitions.
      
      yy_scan_buffer, yy_scan_string, yy_scan_bytes: Leave these unused extern
      function declarations alone.
      
      yypush_buffer_state, yypop_buffer_state, yyswitch_to_buffer, yyget*, yyset*,
      yylex_destroy: Add __attribute__((unused)).
      
      This closes Mantis issue #8.
      c5c5fdc5
    • marko's avatar
      branches/zip: trx_undo_prev_version_build(): Remove a bogus warning · 3ab0d2d1
      marko authored
      about undo_rec possibly being uninitialized.  When trx_undo_get_undo_rec()
      leaves undo_rec uninitialized, both functions will return DB_MISSING_HISTORY
      without dereferencing undo_rec.
      
      This closes Mantis issue #7.
      3ab0d2d1
  9. 06 Feb, 2008 5 commits
    • marko's avatar
      branches/zip: Hide some global variables that were accidentally not hidden · 468d425d
      marko authored
      in r2276.  Now the following symbols will be exported when InnoDB is built
      as a dynamic plugin:
      
      * the virtual method pointer table of class ha_innodb
      * the three variables that MySQL will reference when linking at runtime:
      
      	_mysql_plugin_declarations_
      	_mysql_plugin_interface_version_
      	_mysql_sizeof_struct_st_plugin_
      
      Furthermore, the following symbols are weak globals, to allow us to access
      the built-in InnoDB in the mysqld executable, in case it contains a statically
      linked InnoDB:
      
      	builtin_innobase_plugin
      	innodb_hton_ptr
      468d425d
    • marko's avatar
      branches/zip: Make the changes to pars0grm.c that were promised in r2276. · fe5e0b72
      marko authored
      pars/pars0grm.h: Remove.  The primary file is include/pars0grm.h.
      
      pars/make_bison.sh: Add from trunk.  Somehow, this file was not merged
      at the same time with make_flex.sh.  Add sed magic for replacing the
      file name and for hiding yychars, yynerrs, yylval, and yyparse.
      
      pars/pars0grm.c: Apply the changes made by make_bison.sh.
      fe5e0b72
    • marko's avatar
      branches/zip: Introduce UNIV_INTERN, a linkage specifier for InnoDB-global · 1d1dc31a
      marko authored
      symbols.  Use it for all definitions of non-static variables and functions.
      
      lexyy.c, make_flex.sh: Declare yylex as UNIV_INTERN, not static.  It is
      referenced from pars0grm.c.
      
      Actually, according to
      	nm .libs/ha_innodb.so|grep -w '[ABCE-TVXYZ]'
      the following symbols are still global:
      
      * The vtable for class ha_innodb
      * pars0grm.c: The function yyparse() and the variables yychar, yylval, yynerrs
      
      The required changes to the Bison-generated file pars0grm.c will be addressed
      in a separate commit, which will add a script similar to make_flex.sh.
      
      The class ha_innodb is renamed from class ha_innobase by a #define.  Thus,
      there will be no clash with the builtin InnoDB.  However, there will be some
      overhead for invoking virtual methods of class ha_innodb.  Ideas for making
      the vtable hidden are welcome.  -fvisibility=hidden is not available in GCC 3.
      1d1dc31a
    • marko's avatar
      branches/zip: Make static all declarations in the Flex-generated · 891074b0
      marko authored
      lexical analyzer of the InnoDB SQL parser.
      891074b0
    • marko's avatar
      dc7d5448
  10. 04 Feb, 2008 1 commit
    • marko's avatar
      branches/zip: Apply some fixes suggested by Heikki. · c9087d7d
      marko authored
      btr_cur_pessimistic_update(): Note why the externally stored columns
      of a record on a latched page cannot have been purged.
      
      trx_undo_get_undo_rec(): Clarify that the stack of versions is locked
      all the way down to the purge view.
      
      trx_undo_prev_version_build(): Set *old_vers = NULL also when the record
      could have been purged already.  Add some clarifying comments.
      c9087d7d
  11. 30 Jan, 2008 1 commit
  12. 29 Jan, 2008 1 commit
    • vasil's avatar
      branches/zip: · c80d5da0
      vasil authored
      Require PROCESS privileges instead of SUPER to view INFORMATION_SCHEMA tables.
      
      Suggested by:	Sergei Golubchik <serg@mysql.com> (in a private email,
      		pointed http://bugs.mysql.com/32710)
      
      c80d5da0
  13. 28 Jan, 2008 1 commit
  14. 25 Jan, 2008 3 commits
    • marko's avatar
      branches/zip: Assert in the adaptive hash that the insert buffer B-tree · 993c02cf
      marko authored
      is not indexed.
      
      btr_search_update_hash_ref(), btr_search_drop_page_hash_index(),
      btr_search_build_page_hash_index(), btr_search_update_hash_on_delete(),
      btr_search_update_hash_node_on_insert(), btr_search_update_hash_on_insert(),
      btr_search_validate():
      Assert that hashed blocks do not belong to the insert buffer tree.
      
      btr_search_move_or_delete_hash_entries():
      When invoked on the insert buffer tree, assert that neither block is hashed.
      993c02cf
    • marko's avatar
      branches/zip: Fast index creation: Release locks on system tables before · e1e958d4
      marko authored
      creating indexes.  Lock the user table inside the user transaction.
      
      enum trx_dict_op: Remove TRX_OP_INDEX_MAY_WAIT.
      
      ha_innobase::add_index(): Lock the user tables within prebuilt->trx.
      Commit the data dictionary transaction before creating indexes.
      
      ha_innobase::final_drop_index(): Lock the user table within prebuilt->trx.
      e1e958d4
    • marko's avatar
      branches/zip: Introduce the accessor function dict_index_is_ibuf() · 0a63ed9c
      marko authored
      for determining if an index is the insert buffer B-tree.
      0a63ed9c
  15. 24 Jan, 2008 2 commits
  16. 23 Jan, 2008 2 commits
    • marko's avatar
      branches/zip: When storing a longer prefix of an externally stored column · b52a30e0
      marko authored
      to the undo log, also store the original length of the column, so that the
      changes will be correctly undone in transaction rollback or when fetching
      previous versions of the row.
      
      innodb-zip.test: New file, for tests of the compression.
      
      upd_field_t: Add orig_len, the original length of new_val.
      
      btr_push_update_extern_fields(): Restore the original prefix of the column.
      Add the parameter heap where memory will be allocated if necessary.
      
      trx_undo_rec_get_col_val(): Add the output parameter orig_len.
      
      trx_undo_page_report_modify_ext(): New function: Write an externally
      stored column to the undo log.  This is only called from
      trx_undo_page_report_modify(), and this is the only caller of
      trx_undo_page_fetch_ext().
      
      trx_undo_update_rec_get_update(): Read the original length of the column
      prefix to upd_field->orig_len.
      b52a30e0
    • vasil's avatar
      branches/zip: · 5ee41a15
      vasil authored
       
      Include univ.i in mysql_addons.h because univ.i includes innodb_redefine.h
      which is needed to rename the functions in this file with ibd_ prefix in
      the dynamic InnoDB plugin. Aka the renaming magic, required for the dynamic
      plugin to work.
      
      Approved by:	Marko
      5ee41a15