1. 13 May, 2008 1 commit
    • unknown's avatar
      Bug#35997 Event scheduler seems to let the server crash, if it is embedded. · c83b9a33
      unknown authored
      Fixed build failure depending on events in embedded build.
      
      
      mysql-test/r/create.result:
        Moved test with events to another test file sine embedded build doesn't
        support event scheduler.
      mysql-test/r/events_2.result:
        Moved test with events to another test file sine embedded build doesn't
        support event scheduler.
      mysql-test/r/sp.result:
        Removed 'show privileges' from test as result is depending on 
        if event sheduler is built in.
      mysql-test/t/create.test:
        Moved test with events to another test file sine embedded build doesn't
        support event scheduler.
      mysql-test/t/events_2.test:
        Moved test with events to another test file sine embedded build doesn't
        support event scheduler.
      mysql-test/t/sp.test:
        Removed 'show privileges' from test as result is depending on 
        if event sheduler is built in.
      c83b9a33
  2. 12 May, 2008 5 commits
  3. 10 May, 2008 1 commit
    • unknown's avatar
      Apply InnoDB snapshot innodb-5.1-ss2438. · 7febbe45
      unknown authored
      Addresses the following bugs:
      
        Change the fix for Bug#32440 to show bytes instead of kilobytes in
        INFORMATION_SCHEMA.TABLES.DATA_FREE.
      
        branches/5.1: Fix bug#29507 TRUNCATE shows to many rows effected
        In InnoDB, the row count is only a rough estimate used by SQL
        optimization. InnoDB is now return row count 0 for TRUNCATE operation.
      
        branches/5.1: Fix bug#35537 - Innodb doesn't increment handler_update
        and handler_delete
        Add the calls to ha_statistic_increment() in ha_innobase::delete_row()
        and ha_innobase::update_row().
      
        Fix Bug#36169 create innodb compressed table with too large row size crashed
        Sometimes it is possible that
        row_drop_table_for_mysql(index->table_name, trx, FALSE); is invoked in
        row_create_index_for_mysql() when the index object is freed so copy the
        table name to a safe place beforehand and use the copy.
      
        Fix Bug#36434 ha_innodb.so is installed in the wrong directory
        Change pkglib_LTLIBRARIES with pkgplugin_LTLIBRARIES which has been
        forgotten in this commit: http://lists.mysql.com/commits/40206
      
      
      mysql-test/r/innodb.result:
        Apply InnoDB snapshot innodb-5.1-ss2438.
        
        Revision r2420:
        branches/5.1: Fix bug#29507 TRUNCATE shows to many rows effected
        
        In InnoDB, the row count is only a rough estimate used by SQL
        optimization. InnoDB is now return row count 0 for TRUNCATE operation.
        
        Revision r2421:
        branches/5.1: Fix bug#35537 - Innodb doesn't increment handler_update
        and handler_delete
        
        Add the calls to ha_statistic_increment() in ha_innobase::delete_row()
        and ha_innobase::update_row().
      mysql-test/t/innodb.test:
        Apply InnoDB snapshot innodb-5.1-ss2438.
        
        Revision r2420:
        branches/5.1: Fix bug#29507 TRUNCATE shows to many rows effected
        
        In InnoDB, the row count is only a rough estimate used by SQL
        optimization. InnoDB is now return row count 0 for TRUNCATE operation.
        
        Revision r2421:
        branches/5.1: Fix bug#35537 - Innodb doesn't increment handler_update
        and handler_delete
        
        Add the calls to ha_statistic_increment() in ha_innobase::delete_row()
        and ha_innobase::update_row().
      storage/innobase/Makefile.am:
        Apply InnoDB snapshot innodb-5.1-ss2438.
        
        Revision r2429:
        branches/5.1:
        
        * Use INNODB_CFLAGS insead of cluttering CFLAGS with InnoDB specific
          flags. CFLAGS are used to compile every file in the MySQL source tree.
        
        * Add INNODB_DYNAMIC_CFLAGS to the flags of the dynamic plugin and use
          -prefer-non-pic to make the dynamic plugin faster on i386.
        
        Approved by:	Sunny
        
        
        Revision r2437:
        branches/5.1:
        
        Fix Bug#36434 ha_innodb.so is installed in the wrong directory
        
        Change pkglib_LTLIBRARIES with pkgplugin_LTLIBRARIES which has been
        forgotten in this commit: http://lists.mysql.com/commits/40206
        
        
        
        Revision r2430:
        branches/5.1:
        
        Use CFLAGS instead of INNODB_CFLAGS to avoid having incompatible flags
        inserted in CFLAGS and INNODB_CFLAGS and to avoid compiling mysql and
        innodb with different flags.
        
        Discussed with:	Sunny
        
        
        Revision r2434:
        branches/5.1:
        
        Merge changes from MySQL:
        
          ChangeSet@1.2645, 2007-12-19 13:24:43+00:00, jperkin@chorlton.adsl.perkin.org.uk +14 -0
            Add new pkgplugindir handling to seperate plugins from libraries,
            and allow override for binary distributions.  Extend mysql_config
            to print compiled-in plugin location for third-party plugins to
            use.  Resolves bug#31736.
        
        and
        
          ChangeSet@1.2646, 2008-01-02 13:00:46+00:00, jperkin@chorlton.adsl.perkin.org.uk +11 -0
            Use pkglibdir to simplify pkgplugindir, and fix the path in a
            couple of Makefiles.  Continuation of the fix for bug#31736.
        
        
        
        Revision r2436:
        branches/5.1:
        
        Non-functional white space change in Makefile.am:
        Use tabs for indentation and be consistent about spaces around the
        equal sign.
      storage/innobase/handler/ha_innodb.cc:
        Apply InnoDB snapshot innodb-5.1-ss2438.
        
        Revision r2419:
        branches/5.1:
        
        Change the fix for Bug#32440 to show bytes instead of kilobytes in
        INFORMATION_SCHEMA.TABLES.DATA_FREE.
        
        Suggested by:	Domas Mituzas <domas@mysql.com>
        Approved by:	Heikki
        
        
        
        Revision r2435:
        branches/5.1:
        
        Merge change from MySQL:
        
          ChangeSet@1.2563, 2008-03-18 19:42:04+04:00, gluh@mysql.com +1 -0
            Bug#35406 5.1-opt crashes on select from I_S.REFERENTIAL_CONSTRAINTS
            added intialization of f_key_info.referenced_key_name for the case when
            referenced table is dropped
        
        
        
        Revision r2420:
        branches/5.1: Fix bug#29507 TRUNCATE shows to many rows effected
        
        In InnoDB, the row count is only a rough estimate used by SQL
        optimization. InnoDB is now return row count 0 for TRUNCATE operation.
        
        Revision r2421:
        branches/5.1: Fix bug#35537 - Innodb doesn't increment handler_update
        and handler_delete
        
        Add the calls to ha_statistic_increment() in ha_innobase::delete_row()
        and ha_innobase::update_row().
      storage/innobase/plug.in:
        Apply InnoDB snapshot innodb-5.1-ss2438.
        
        Revision r2429:
        branches/5.1:
        
        * Use INNODB_CFLAGS insead of cluttering CFLAGS with InnoDB specific
          flags. CFLAGS are used to compile every file in the MySQL source tree.
        
        * Add INNODB_DYNAMIC_CFLAGS to the flags of the dynamic plugin and use
          -prefer-non-pic to make the dynamic plugin faster on i386.
        
        Approved by:	Sunny
        
        
        Revision r2432:
        branches/5.1:
        
        Non-functional change: use tabs for indentation in plug.in.
        
        
        Revision r2431:
        branches/5.1:
        
        Fix r2430, it should be CFLAGS="$CFLAGS ..." not CFLAGS="..."
        
        
        Revision r2430:
        branches/5.1:
        
        Use CFLAGS instead of INNODB_CFLAGS to avoid having incompatible flags
        inserted in CFLAGS and INNODB_CFLAGS and to avoid compiling mysql and
        innodb with different flags.
        
        Discussed with:	Sunny
        
        
        Revision r2433:
        branches/5.1:
        
        Add vim modeline to hint it that plug.in is a config file
        so it can be colorized.
      storage/innobase/row/row0mysql.c:
        Apply InnoDB snapshot innodb-5.1-ss2438.
        
        Revision r2422:
        branches/5.1:
        
        Fix Bug#36169 create innodb compressed table with too large row size crashed
        
        Sometimes it is possible that
        row_drop_table_for_mysql(index->table_name, trx, FALSE); is invoked in
        row_create_index_for_mysql() when the index object is freed so copy the
        table name to a safe place beforehand and use the copy.
        
        Approved by:	Sunny
      storage/innobase/row/row0sel.c:
        Apply InnoDB snapshot innodb-5.1-ss2438.
        
        Revision r2388:
        branches/5.1:
        
        Swap the order in which mysql_thd, mysql_query_str and *mysql_query_str
        are checked for non-NULL.
        
        Suggested by:	Marko
      7febbe45
  4. 09 May, 2008 6 commits
    • unknown's avatar
      Merge hh-nb.hungers:/work/merge/mysql-5.1-bugteam · 484d8d33
      unknown authored
      into  hh-nb.hungers:/work/merge/mysql-5.1-36522
      
      
      484d8d33
    • unknown's avatar
      Bug#35997 Event scheduler seems to let the server crash, if it is embedded. · 5908a32e
      unknown authored
      Enable previously disabled test cases which were tested against
      the embedded build. The test cases are modified so that they require
      non-embedded build.
      
      
      mysql-test/t/disabled.def:
        Re-enabled event_scheduler_basic and event_scheduler_func since
        these tests aren't suppose to work in embedded build anyway.
      mysql-test/t/event_scheduler_basic.test:
        Require that this test isn't run on embedded build
      mysql-test/t/event_scheduler_func.test:
        Require that this test isn't run on embedded build
      5908a32e
    • unknown's avatar
      Bug#35997 Event scheduler seems to let the server crash, if it is embedded. · 9a6fdf1f
      unknown authored
      Missing file: include/is_embedded.inc
      
      This file was left out of the initial changeset by misstake.
      
      
      mysql-test/include/is_embedded.inc:
        Missing file patch: include/is_embedded.inc
      9a6fdf1f
    • unknown's avatar
      Merge adventure.(none):/home/thek/Development/cpp/bug35997/my51-bug35997 · ede5ee9b
      unknown authored
      into  adventure.(none):/home/thek/Development/cpp/mysql-5.1-bugteam
      
      
      ede5ee9b
    • unknown's avatar
      Bug#35997 Event scheduler seems to let the server crash, if it is embedded. · 3cf9e6eb
      unknown authored
      The event scheduler was not designed to work in embedded mode. This
      patch disables and excludes the event scheduler when the server is
      compiled for embedded build.
      
      
      libmysqld/Makefile.am:
        Reduce the amount of event code in an embedded build.
      mysql-test/t/events_trans.test:
        Disable test if run in embedded mode.
      sql/Makefile.am:
        Introduce definition HAVE_EVENT_SCHEDULER and one new source file.
      sql/event_data_objects.cc:
        Refactor Event_parse_data to new file.
      sql/event_data_objects.h:
        Refactor Event_parse_data to new file.
        Move global definitions to new file.
      sql/event_queue.cc:
        Move all parsed items to Event_parse_data for easier modularization.
      sql/events.cc:
        Move all parsed items to Event_parse_data for easier modularization.
      sql/mysqld.cc:
        Disable the event schedular subsystem if the server is compiled in
        embedded mode.
      sql/set_var.cc:
        Disable the event schedular subsystem if the server is compiled in
        embedded mode.
      sql/set_var.h:
        Disable the event schedular subsystem if the server is compiled in
        embedded mode.
      sql/sql_db.cc:
        Disable the event schedular subsystem if the server is compiled in
        embedded mode.
      sql/sql_parse.cc:
        Disable the event schedular subsystem if the server is compiled in
        embedded mode.
      sql/sql_show.cc:
        Disable the event schedular subsystem if the server is compiled in
        embedded mode.
      sql/sql_test.cc:
        Disable the event schedular subsystem if the server is compiled in
        embedded mode.
      sql/sql_yacc.yy:
        Only include event-code needed for parsing to reduce impact on embedded
        build.
        Move all constants to Event_parse_data class.
      mysql-test/r/events_embedded.result:
        Add test case to make sure the 'event_scheduler' can't be activated
        in embedded mode.
      mysql-test/r/is_embedded.require:
        Add test case to make sure the 'event_scheduler' can't be activated
        in embedded mode.
      mysql-test/t/events_embedded.test:
        Add test case to make sure the 'event_scheduler' can't be activated
        in embedded mode.
      sql/event_parse_data.cc:
        New file. Extracted Event_parse data into a new file.
      sql/event_parse_data.h:
        New file. Extracted Event_parse data into a new file.
      3cf9e6eb
    • unknown's avatar
      Bug #34297: MySQL Server crashes when processing large table · c3afb478
      unknown authored
      Remove optimizations on innobase/mem/* to avoid apparent compiler bug which
      causes memory overruns.  See also bug 19424, and probably bug 36366.
      
      This is done in 5.1+; 5.0 already has this workaround in place.
      
      
      storage/innobase/CMakeLists.txt:
        Remove optimizations on innobase/mem/* to avoid apparent compiler bug which
        causes memory overruns.  See bug 34297, bug 19424, and probably bug 36366.
      c3afb478
  5. 08 May, 2008 5 commits
    • unknown's avatar
      Fixes for bug#36522. · db03bda0
      unknown authored
      
      BitKeeper/etc/ignore:
        Added mysql-test/bug36522-64.tar mysql-test/bug36522.tar mysql-test/t.log mysql-test/tps.log to the ignore list
      db03bda0
    • unknown's avatar
      Bug#32575: Parse error of stmt with extended comments on slave side · f72c7e0a
      unknown authored
      test change for 5.1+ (show create view adds two columns in 5.1)
      
      Patch only for 5.1+
      
      
      mysql-test/suite/rpl/r/rpl_sp.result:
        restored the original 5.1 test result
      mysql-test/suite/rpl/r/rpl_view.result:
        Bug#32575: Parse error of stmt with extended comments on slave side
        
        Updated test result for show create view in 5.0 to 5.1
      f72c7e0a
    • unknown's avatar
      Merge client-10-129-10-147.upp.off.mysql.com:/Users/mattiasj/clones/topush-50-bugteam · 266eb2bd
      unknown authored
      into  client-10-129-10-147.upp.off.mysql.com:/Users/mattiasj/clones/topush-51-bugteam
      
      
      mysql-test/suite/rpl/r/rpl_sp.result:
        Auto merged
      mysql-test/suite/rpl/t/rpl_view.test:
        Auto merged
      sql/sql_view.cc:
        This was already fixed in 5.1
      mysql-test/suite/rpl/r/rpl_view.result:
        manual merge
      266eb2bd
    • unknown's avatar
      Merge client-10-129-10-147.upp.off.mysql.com:/Users/mattiasj/clones/bug32575-50-bugteam · ab6e91cf
      unknown authored
      into  client-10-129-10-147.upp.off.mysql.com:/Users/mattiasj/clones/topush-50-bugteam
      
      
      ab6e91cf
    • unknown's avatar
      Bug#32575 - Parse error of stmt with extended comments on slave side · a6f7fa35
      unknown authored
      Problem was that mysql_create_view did not remove all comments characters
      when writing to binlog, resulting in parse error of stmt on slave side.
      
      Solution was to use the recreated select clause
      and add a generated CHECK OPTION clause if needed.
      
      
      mysql-test/r/rpl_sp.result:
        Bug#32575 - Parse error of stmt with extended comments on slave side
        
        Updated test result
      mysql-test/r/rpl_view.result:
        Bug#32575 - Parse error of stmt with extended comments on slave side
        
        Updated test result
      mysql-test/t/rpl_view.test:
        Bug#32575 - Parse error of stmt with extended comments on slave side
        
        Added test case
      sql/sql_view.cc:
        Bug#32575 - Parse error of stmt with extended comments on slave side
        
        Problem was that mysql_create_view did not remove all comments characters
        when writing to binlog, resulting in parse error of stmt on slave side.
        
        Solution was to use the recreated select clause and generate
        'WITH {LOCAL|CASCADED} CHECK OPTION'.
      a6f7fa35
  6. 07 May, 2008 6 commits
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-bugteam · 41a7fa62
      unknown authored
      into  mysql.com:/Users/davi/mysql/mysql-5.1-bugteam
      
      
      41a7fa62
    • unknown's avatar
      Merge riska.(none):/home/sven/bk/b36379-enable_test/gca · 612a3ab4
      unknown authored
      into  riska.(none):/home/sven/bkroot/5.1-bugteam
      
      
      mysql-test/suite/rpl/t/disabled.def:
        Auto merged
      612a3ab4
    • unknown's avatar
      BUG#36398: 'DROP TEMPORARY TABLE' in ROLLBACK transaction causes transaction to be logged · e1a2bb92
      unknown authored
      Temporarily checking in an incorrect test case. Rationale: the impact of
      this bug is negligible (it's almost a feature request). We need 5.1 to be
      stable, and making a real fix is a bit risky. So the fix is postponed
      to 6.0.
      
      
      mysql-test/suite/rpl/r/rpl_ddl.result:
        Updated result file.
      mysql-test/suite/rpl/t/disabled.def:
        Enabling test case rpl_ddl.test
      e1a2bb92
    • unknown's avatar
      BUG#36379: please re-enable rpl_innodb_bug28430 · fcec0b01
      unknown authored
      The test suite/rpl/t/rpl_innodb_bug28430.test was disabled because of
      BUG#32247, but not re-enabled when BUG#32247 was fixed. I've re-enabled
      it. The test and result file needed to be updated too.
      
      
      mysql-test/suite/rpl/r/rpl_innodb_bug28430.result:
        Updated result file.
      mysql-test/suite/rpl/t/disabled.def:
        Enabled the rpl_innodb_bug28430 test case.
      mysql-test/suite/rpl/t/rpl_innodb_bug28430.test:
        show slave status is not needed since we have select count(*)
      fcec0b01
    • unknown's avatar
      BUG#35850 "Performance regression in 5.1.23/5.1.24" · 87e50982
      unknown authored
      - Disable the "prefer full scan on clustered primary key over full scan
        of any secondary key" rule introduced by BUG#35850.
      - Update test results accordingly 
      (bk trigger: file this for BUG#35850)
      
      
      mysql-test/r/innodb.result:
        BUG#35850 "Performance regression in 5.1.23/5.1.24"
        - Update test results
      mysql-test/r/innodb_mysql.result:
        BUG#35850 "Performance regression in 5.1.23/5.1.24"
        - Testcase
        - Update test results
      mysql-test/r/join_outer_innodb.result:
        BUG#35850 "Performance regression in 5.1.23/5.1.24"
        - Update test results
      mysql-test/t/innodb_mysql.test:
        BUG#35850 "Performance regression in 5.1.23/5.1.24"
        - Testcase
      sql/sql_select.cc:
        BUG#35850 "Performance regression in 5.1.23/5.1.24"
        - Disable the "prefer full scan on clustered primary key over full scan
          of any secondary key" rule introduced by BUG#35850.
      87e50982
    • unknown's avatar
      Re-enable test case thread_cache_size_func as Bug 35988 · 0cf1439c
      unknown authored
      has been fixed and closed.
      
      
      mysql-test/t/disabled.def:
        Re-enable test case as bug has been fixed.
      0cf1439c
  7. 06 May, 2008 10 commits
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-bugteam · d1361c86
      unknown authored
      into  mysql.com:/Users/davi/mysql/mysql-5.1-bugteam
      
      
      d1361c86
    • unknown's avatar
      Merge mysql.com:/Users/davi/mysql/bugs/34108-5.1 · e5491d73
      unknown authored
      into  mysql.com:/Users/davi/mysql/mysql-5.1-bugteam
      
      
      e5491d73
    • unknown's avatar
      Bug#34108 "Drop function" (test "storedproc_06") returns different message (all handlers) · b54b7c65
      unknown authored
      The problem is that the patch for Bug 33464 didn't update the
      results of the test cases which caused the problem.
      
      Dropping a stored routine also revoke privileges for all users
      on the stored routine and errors about missing grants are converted
      into warnings. Before Bug 33464 such errors could be incorrectly
      returned to the user, which would later trigger a assertion due to
      multiple errors being set.
      
      
      mysql-test/suite/funcs_1/r/innodb_storedproc_06.result:
        Update test case result, grants were already dropped.
      mysql-test/suite/funcs_1/r/memory_storedproc_06.result:
        Update test case result, grants were already dropped.
      mysql-test/suite/funcs_1/r/myisam_storedproc_06.result:
        Update test case result, grants were already dropped.
      mysql-test/suite/funcs_1/r/ndb_storedproc_06.result:
        Update test case result, grants were already dropped.
      mysql-test/suite/funcs_1/storedproc/storedproc_06.inc:
        Add comment regarding bug revealed by test case.
      mysql-test/suite/funcs_1/t/disabled.def:
        Re-enable test cases, bug has been fixed.
      b54b7c65
    • unknown's avatar
      Merge kpettersson@bk-internal.mysql.com:/home/bk/mysql-5.1-bugteam · 67809a7a
      unknown authored
      into  adventure.(none):/home/thek/Development/cpp/mysql-5.1-bugteam
      
      
      67809a7a
    • unknown's avatar
      Adjusted a filaing test and re-enabled it. · 726817e6
      unknown authored
      
      mysql-test/r/query_cache_wlock_invalidate_func.result:
        Adjusted test. Query cache doesn't support multiple statements and thus
        using a delimiter change won't work.
      mysql-test/t/disabled.def:
        Enabled the query_cache_wlock_invalidate_func test.
      mysql-test/t/query_cache_wlock_invalidate_func.test:
        Adjusted test. Query cache doesn't support multiple statements and thus
        using a delimiter change won't work.
      726817e6
    • unknown's avatar
      Silence warning due to copying the address of a string constant · 2ac9a30f
      unknown authored
      into a non-const string pointer.
      
      
      client/mysqltest.c:
        Don't use a constant string as val may be modified later.
      2ac9a30f
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-bugteam · 9a472a21
      unknown authored
      into  mysql.com:/Users/davi/mysql/mysql-5.1-bugteam
      
      
      9a472a21
    • unknown's avatar
      Merge noir.wlan.koehntopp.de:/misc/mysql/mysql-5.0-bugteam · a7028d97
      unknown authored
      into  noir.wlan.koehntopp.de:/misc/mysql/mysql-5.1-bugteam
      
      
      mysys/my_static.c:
        Auto merged
      mysys/my_static.h:
        Auto merged
      a7028d97
    • unknown's avatar
      revert the push of bug 35616. · 65a310fe
      unknown authored
      65a310fe
    • unknown's avatar
      Bug#35616: memory overrun on 64-bit linux on setting large values for keybuffer-size · 55c336fd
      unknown authored
      We could allocate chunks larger than 4GB, but did our size-accounting in 32-bit
      values. This could lead to spurious warnings, inaccurate accounting, and, in
      theory, data loss.
      
      Affected: 64-bit platforms. Debug-build (with safemalloc). At least one buffer
      larger than 4GB. For potential data loss, a re-alloc on such a buffer would be
      necessary.
      
      
      mysys/my_static.c:
        Make memory-accounting 64-bit safe.
      mysys/my_static.h:
        Make memory-accounting 64-bit safe.
        Move in struct for better alignment when 64-bit.
      55c336fd
  8. 05 May, 2008 5 commits
  9. 02 May, 2008 1 commit
    • unknown's avatar
      Bug#36031 Test funcs_1.<engine>_views failing on Windows · 7ecc40cc
      unknown authored
      The problem is a hack in mysqltest.c::append_field that modifies
      the exponential notation of floating point numbers by removing a
      zero after the the symbol 'e' (eg: 00001.2e+018 is converted to
      00001.2e+18) but does not take into account the zerofill affect
      in the start of the string.
      
      The solution is to check if the field was zero filled and insert
      a zero at the start of the string if a zero after the exponential
      notation symbol is removed.
      
      
      client/mysqltest.c:
        Preserve zerofill affect when removing a zero after
        the exponential notation symbol.
      7ecc40cc