1. 31 Jul, 2007 2 commits
  2. 30 Jul, 2007 3 commits
    • unknown's avatar
      (Pushing for Andrei) · 47965f56
      unknown authored
      Merge magare.gmz:/home/kgeorge/mysql/work/B27417-5.0-opt
      into  magare.gmz:/home/kgeorge/mysql/work/B27417-5.1-opt
      
      
      mysql-test/suite/binlog/r/binlog_stm_mix_innodb_myisam.result:
        Auto merged
      sql/ha_ndbcluster.cc:
        Auto merged
      sql/handler.cc:
        Auto merged
      sql/handler.h:
        Auto merged
      sql/sql_class.cc:
        Auto merged
      sql/sql_class.h:
        Auto merged
      sql/sql_table.cc:
        Auto merged
      mysql-test/extra/binlog_tests/mix_innodb_myisam_binlog.test:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/log.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/set_var.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/sp_head.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/sql_delete.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/sql_insert.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/sql_load.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/sql_parse.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      sql/sql_update.cc:
        merge of bug 27471 from 5.0-opt to 5.1-opt
      47965f56
    • unknown's avatar
      (pushing for Andrei) · 817a05a6
      unknown authored
      Bug #27417 thd->no_trans_update.stmt lost value inside of SF-exec-stack
        
      Once had been set the flag might later got reset inside of a stored routine 
      execution stack.
      The reason was in that there was no check if a new statement started at time 
      of resetting.
      The artifact affects most of binlogable DML queries. Notice, that multi-update 
      is wrapped up within
        bug@27716 fix, multi-delete bug@29136.
        
      Fixed with saving parent's statement flag of whether the statement modified 
      non-transactional table, and unioning (merging) the value with that was gained 
      in mysql_execute_command.
        
      Resettling thd->no_trans_update members into thd->transaction.`member`;
      Asserting code;
      Effectively the following properties are held.
        
      1. At the end of a substatement thd->transaction.stmt.modified_non_trans_table
         reflects the fact if such a table got modified by the substatement.
         That also respects THD::really_abort_on_warnin() requirements.
      2. Eventually thd->transaction.stmt.modified_non_trans_table will be computed as
         the union of the values of all invoked sub-statements.
         That fixes this bug#27417;
      
      Computing of thd->transaction.all.modified_non_trans_table is refined to base to 
      the stmt's value for all the case including insert .. select statement which 
      before the patch had an extra issue bug@28960.
      Minor issues are covered with mysql_load, mysql_delete, and binloggin of insert in
      to temp_table select. 
        
      The supplied test verifies limitely, mostly asserts. The ultimate testing is defered
      for bug@13270, bug@23333.
      
      
      mysql-test/r/mix_innodb_myisam_binlog.result:
        results changed
      mysql-test/t/mix_innodb_myisam_binlog.test:
        regression test incl the related bug#28960.
      sql/ha_ndbcluster.cc:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/handler.cc:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/handler.h:
        new member added
      sql/log.cc:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/set_var.cc:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/sp_head.cc:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
        
        and saving and merging stmt's flag at the end of a substatement.
      sql/sql_class.cc:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/sql_class.h:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/sql_delete.cc:
        correcting basic delete incl truncate branch and multi-delete queries to set
        stmt.modified_non_trans_table;
        optimization to set the flag at the end of per-row loop;
        multi-delete still has an extra issue similar to bug#27716 of multi-update 
        - to be address with bug_29136 fix.
      sql/sql_insert.cc:
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/sql_load.cc:
        eliminating a separate issue where the stmt flag was saved and re-stored after 
        write_record that actually could change it and the change would be lost but 
        should remain permanent;
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      sql/sql_parse.cc:
        initialization to transaction.stmt.modified_non_trans_table at the common part 
        of all types of statements processing - mysql_execute_command().
      sql/sql_table.cc:
        moving the reset up to the mysql_execute_command() caller
      sql/sql_update.cc:
        correcting update query case (multi-update part of the issues covered by other 
        bug#27716 fix)
        thd->transaction.{all,stmt}.modified_non_trans_table
        instead of
        thd->no_trans_update.{all,stmt}
      817a05a6
    • unknown's avatar
      Moved the DBUG_ASSERT from bug 28983 to · 137a2fb6
      unknown authored
      a place where it would not obstruct
      correct multithreading.
      
      137a2fb6
  3. 29 Jul, 2007 3 commits
    • unknown's avatar
      disabled.def: · a26d2bab
      unknown authored
        ddl_i18n_koi8r.test has been disabled to ignore bug #30120.
      
      
      mysql-test/t/disabled.def:
        ddl_i18n_koi8r.test has been disabled to ignore bug #30120.
      a26d2bab
    • unknown's avatar
      sp_head.cc: · da1c98da
      unknown authored
        Post-merge fix.
      
      
      sql/sp_head.cc:
        Post-merge fix.
      da1c98da
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt · 8056fcc7
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.1-opt
      
      
      mysql-test/r/query_cache.result:
        Auto merged
      mysql-test/r/view.result:
        Auto merged
      mysql-test/t/query_cache.test:
        Auto merged
      sql/sql_show.cc:
        Auto merged
      mysql-test/t/view.test:
        Merge with 5.0-opt.
      sql/sp_head.cc:
        Merge with 5.0-opt.
      8056fcc7
  4. 28 Jul, 2007 15 commits
    • unknown's avatar
      Merge epotemkin@bk-internal.mysql.com:/home/bk/mysql-5.0-opt · 90a92562
      unknown authored
      into  moonbone.local:/mnt/gentoo64/work/29856-bug-5.0-opt-mysql
      
      
      sql/sp_head.cc:
        Auto merged
      mysql-test/r/query_cache.result:
        SCCS merged
      mysql-test/t/query_cache.test:
        SCCS merged
      90a92562
    • unknown's avatar
      Merge gshchepa@bk-internal.mysql.com:/home/bk/mysql-5.0-opt · 0c161e15
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.0-opt
      
      0c161e15
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt · d17eee3f
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.1-opt
      
      
      mysql-test/r/sp.result:
        Auto merged
      mysql-test/t/sp.test:
        Auto merged
      sql/sql_base.cc:
        Merge with 5.0-opt.
      d17eee3f
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt-29834 · 95324889
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.0-opt
      
      
      sql/sql_base.cc:
        Auto merged
      mysql-test/r/sp.result:
        Merge with local tree.
      mysql-test/t/sp.test:
        Merge with local tree.
      95324889
    • unknown's avatar
      Fixed bug #29834. · 6ed4e3dd
      unknown authored
      Using view columns by their names during an execution of
      a prepared SELECT statement or a SELECT statement inside
      a SP caused a memory leak.
      
      
      sql/sql_base.cc:
        Fixed bug #29834.
        The find_field_in_view function has been modified to
        use the execution memory root for the Item_direct_view_ref
        objects allocation at non-first executions of
        a PS/SP instead of the statement memory.
      mysql-test/t/sp.test:
        Updated test case for bug #29834.
      mysql-test/r/sp.result:
        Updated test case for bug #29834.
      6ed4e3dd
    • unknown's avatar
      Merge epotemkin@bk-internal.mysql.com:/home/bk/mysql-5.0-opt · c8d0796c
      unknown authored
      into  moonbone.local:/mnt/gentoo64/work/30020-bug-5.0-opt-mysql
      
      c8d0796c
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt · 31c3d61f
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.1-opt
      
      31c3d61f
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.1 · 047be65c
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.1-opt
      
      047be65c
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0 · 439f5d0f
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.0-opt
      
      439f5d0f
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt · 214732e5
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.1-opt
      
      214732e5
    • unknown's avatar
      Bug#30020: Insufficient check led to a wrong info provided by the · 8306d3dc
      unknown authored
      information schema table.
      
      The get_schema_views_record() function fills records in the view table of
      the informations schema with data about given views. Among other info
      the is_updatable flag is set. But the check whether the view is updatable or
      not wasn't covering all cases thus sometimes providing wrong info.
      This might led to a user confusion.
      
      Now the get_schema_views_record function additionally calls to the 
      view->can_be_merge() function to find out whether the view can be updated or
      not.
      
      
      mysql-test/t/view.test:
        Added a test case for the bug#30020: Insufficient check led to a wrong info
        provided by the information schema table.
      mysql-test/r/view.result:
        Added a test case for the bug#30020: Insufficient check led to a wrong info
        provided by the information schema table.
      sql/sql_show.cc:
        Bug#30020: Insufficient check led to a wrong info provided by the
        information schema table.
        Now the get_schema_views_record function additionally calls to the 
        view->can_be_merge() function to find out whether the view can be updated or
        not.
      8306d3dc
    • unknown's avatar
      Bug#29856: Insufficient buffer space led to a server crash. · a1e18414
      unknown authored
      The subst_spvars function is used to create query string with SP variables 
      substituted with their values. This string is used later for the binary log
      and for the query cache. The problem is that the
      query_cache_send_result_to_client function requires some additional space
      after the query to store database name and query cache flags. This 
      space wasn't reserved by the subst_spvars function which led to a memory
      corruption and crash.
      
      Now the subst_spvars function reserves additional space for the query cache.
      
      
      mysql-test/t/query_cache.test:
        Added a test case for the bug#29856: Insufficient buffer space led to a server crash.
      mysql-test/r/query_cache.result:
        Added a test case for the bug#29856: Insufficient buffer space led to a server crash.
      sql/sp_head.cc:
        Bug#29856: Insufficient buffer space led to a server crash.
        Now the subst_spvars function reserves additional space for the query cache.
      a1e18414
    • unknown's avatar
      Merge abotchkov@bk-internal.mysql.com:/home/bk/mysql-5.1-opt · 1bb150e2
      unknown authored
      into  mysql.com:/home/hf/work/29878/my51-29878
      
      1bb150e2
    • unknown's avatar
      rpl_session_var.test fixed to not depend on mysql_test_run parameters · 02736989
      unknown authored
      
      mysql-test/suite/rpl/r/rpl_session_var.result:
        result fixed
      mysql-test/suite/rpl/t/rpl_session_var.test:
        test fixed
      02736989
    • unknown's avatar
      show_check.result: · 7a72380d
      unknown authored
        Addendum to thr fix for bug 30000:
        show procedure/function code defined only in debug builds
      show_check.test:
        Addendum to thr fix for bug 30000: 
        show procedure/function code defined only in debug builds
      
      
      mysql-test/t/show_check.test:
        Addendum to thr fix for bug 30000: 
        show procedure/function code defined only in debug builds
      mysql-test/r/show_check.result:
        Addendum to thr fix for bug 30000:
        show procedure/function code defined only in debug builds
      7a72380d
  5. 27 Jul, 2007 14 commits
    • unknown's avatar
      Merge bk@192.168.21.1:mysql-5.0-opt · 4f1a6146
      unknown authored
      into  mysql.com:/home/hf/work/29878/my50-29878
      
      4f1a6146
    • unknown's avatar
      Merge bk@192.168.21.1:mysql-5.1-opt · b77a4a7b
      unknown authored
      into  mysql.com:/home/hf/work/29878/my51-29878
      
      b77a4a7b
    • unknown's avatar
      Merge mysql.com:/home/hf/work/29878/my50-29878 · 9e33e325
      unknown authored
      into  mysql.com:/home/hf/work/29878/my51-29878
      
      
      mysql-test/suite/rpl/r/rpl_session_var.result:
        Auto merged
      mysql-test/suite/rpl/t/rpl_session_var.test:
        Auto merged
      sql/item.cc:
        Auto merged
      sql/item.h:
        Auto merged
      sql/item_strfunc.h:
        Auto merged
      9e33e325
    • unknown's avatar
      Merge gkodinov@bk-internal.mysql.com:/home/bk/mysql-5.1-opt · 2275dc0e
      unknown authored
      into  magare.gmz:/home/kgeorge/mysql/autopush/B30000-5.1-opt
      
      2275dc0e
    • unknown's avatar
      Bug #30000: SHOW commands once again ends up in the · 4f88fafd
      unknown authored
       slow query log
      Disable the SHOW commands to appear in the show query
      log.
      Update the commands type array.
      
      
      mysql-test/r/show_check.result:
        Bug #30000: test case
      mysql-test/t/show_check.test:
        Bug #30000: test case
      sql/sql_parse.cc:
        Bug #30000: skip SHOW commands from the
        slow query log.
      4f88fafd
    • unknown's avatar
      fixing the 5.1-opt merge of the fix for bug 29571: · 14bc8d09
      unknown authored
      cannot check the statments in the binlog for row based 
      replication.
      
      14bc8d09
    • unknown's avatar
      Bug #29878 Garbage data generation when executing SESSION_USER() on a slave. · e35c8aa4
      unknown authored
      Item_func_user doesn't calculate anything in it's val_str() method,
      just returns saved str_value.
      Though Item::save_in_field method can destroy str_value, relying on
      val_str() return. As a result we get the garbage stored in field.
      
      We cannot use Item::save_in_field implementation for Item_func_user,
      reimplement it in simpler way.
      
      
      mysql-test/r/rpl_session_var.result:
        Bug #29878 Garbage data generation when executing SESSION_USER() on a slave.
        
        test result
      mysql-test/t/rpl_session_var.test:
        Bug #29878 Garbage data generation when executing SESSION_USER() on a slave.
        
        test case
      sql/item.cc:
        Bug #29878 Garbage data generation when executing SESSION_USER() on a slave.
        
        duplicating code moved to Item::save_str_in_field
      sql/item.h:
        Bug #29878 Garbage data generation when executing SESSION_USER() on a slave.
        
        duplicating code moved to Item::save_str_in_field
      sql/item_strfunc.h:
        Bug #29878 Garbage data generation when executing SESSION_USER() on a slave.
        
        Item_func_user::save_in_field implemented as simple storing str_value
      e35c8aa4
    • unknown's avatar
    • unknown's avatar
      5.0-opt -> 5.1-opt merge of the test case for bug 29571: · 28043f17
      unknown authored
       - reset the logs before and after the test.
       - cleanup from the previous tests : use the correct schema.
      
      
      mysql-test/extra/rpl_tests/rpl_insert_delayed.test:
        5.0-opt -> 5.1-opt merge of the test case for bug 29571.
      mysql-test/suite/rpl/r/rpl_stm_insert_delayed.result:
        5.0-opt -> 5.1-opt merge of the test case for bug 29571.
      28043f17
    • unknown's avatar
      Merge gkodinov@bk-internal.mysql.com:/home/bk/mysql-5.1-opt · 6f8f899a
      unknown authored
      into  magare.gmz:/home/kgeorge/mysql/autopush/B30036-5.1-opt
      
      6f8f899a
    • unknown's avatar
      Merge mysql.com:/home/svoj/devel/mysql/BUG29957/mysql-5.0-engines · 04727a4c
      unknown authored
      into  mysql.com:/home/svoj/devel/mysql/BUG29957/mysql-5.1-engines
      
      
      storage/myisam/mi_extra.c:
        Auto merged
      mysql-test/r/alter_table.result:
        Manual merge
      mysql-test/t/alter_table.test:
        Manual merge
      04727a4c
    • unknown's avatar
      Merge svojtovich@bk-internal.mysql.com:/home/bk/mysql-5.0-engines · a52fe603
      unknown authored
      into  mysql.com:/home/svoj/devel/mysql/BUG29957/mysql-5.0-engines
      
      a52fe603
    • unknown's avatar
      BUG#29957 - alter_table.test fails · e011d300
      unknown authored
      INSERT/DELETE/UPDATE followed by ALTER TABLE within LOCK TABLES
      may cause table corruption on Windows.
      
      That happens because ALTER TABLE writes outdated shared state
      info into index file.
      
      Fixed by removing obsolete workaround.
      
      Affects MyISAM tables on Windows only.
      
      
      myisam/mi_extra.c:
        On windows when mi_extra(HA_EXTRA_PREPARE_FOR_DELETE) is called,
        we release external lock and close index file. If we're in LOCK
        TABLES, MyISAM state info doesn't get updated until UNLOCK TABLES.
        
        That means when we release external lock and we're in LOCK TABLES,
        we may write outdated state info.
        
        As SQL layer closes all table instances, we do not need this
        workaround anymore.
      mysql-test/r/alter_table.result:
        A test case for BUG#29957.
      mysql-test/t/alter_table.test:
        A test case for BUG#29957.
      e011d300
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt · ac2cd6d6
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.1-opt
      
      
      sql/sql_insert.cc:
        Auto merged
      BitKeeper/deleted/.del-bdb.result:
        Auto merged
      BitKeeper/deleted/.del-ha_berkeley.cc:
        Auto merged
      BitKeeper/deleted/.del-rpl_insert_delayed.result:
        Auto merged
      mysql-test/extra/rpl_tests/rpl_insert_delayed.test:
        Auto merged
      sql/sql_select.cc:
        Merge with 5.0-opt.
      sql/table.cc:
        Merge with 5.0-opt.
      ac2cd6d6
  6. 26 Jul, 2007 3 commits
    • unknown's avatar
      Merge spetrunia@bk-internal.mysql.com:/home/bk/mysql-5.0-opt · d26eb0b3
      unknown authored
      into  mysql.com:/home/psergey/mysql-5.0-bdb-fix
      
      d26eb0b3
    • unknown's avatar
      BUG#28591: make the fix work for BDB tables too: · 532a96cb
      unknown authored
       - make ha_berkeley::cmp_ref() take into account that auto-generated PKs 
         are stored in LSB-first order. 
       - Remove the temporary code that made the bugfix work for innodb only 
      
      
      mysql-test/r/bdb.result:
        Adjust test-results.
      sql/ha_berkeley.cc:
        BUG#28591: make the fix work for BDB tables too:
         - make ha_berkeley::cmp_ref() take into account that auto-generated PKs 
           are stored in LSB-first order.
      sql/sql_select.cc:
        BUG#28591: Remove "innodb only" clause as the fix now works for BDB too
      sql/table.cc:
        BUG#28591: Remove "innodb only" clause as the fix now works for BDB too
      532a96cb
    • unknown's avatar
      Bug #30036: SHOW TABLE TYPES causes the debug client to crash · 97a31041
      unknown authored
      SHOW TABLE TYPES is a (to be deprecated) synonym for 
      SHOW STORAGE ENGINES.
      Fixed to use the right schema table in addition to 
      issuing a warning.
      
      
      mysql-test/r/show_check.result:
        Bug #30036: test case
      mysql-test/t/show_check.test:
        Bug #30036: test case
      sql/sql_yacc.yy:
        Bug #30036: use the right schema table
      97a31041