1. 06 Oct, 2008 1 commit
    • Alexey Botchkov's avatar
      Bug#38083 Error-causing row inserted into partitioned table despite error · afcc4e58
      Alexey Botchkov authored
            
          problems are located in the sql_partition.cc where functions calculation
          partition_id don't expect error returned from item->val_int().
          Fixed by adding checks to these functions.
          Note  - it tries to fix more problems than just the reported bug.
            
      per-file comments:
      modified:
        mysql-test/r/partition.result
          Bug#38083 Error-causing row inserted into partitioned table despite error
              test result
        mysql-test/t/partition.test
          Bug#38083 Error-causing row inserted into partitioned table despite error
              test case
        sql/opt_range.cc
          Bug#38083 Error-causing row inserted into partitioned table despite error
              get_part_id() call fixed
        sql/partition_info.h
          Bug#38083 Error-causing row inserted into partitioned table despite error
              get_subpart_id_func interface changed. 
        sql/sql_partition.cc
          Bug#38083 Error-causing row inserted into partitioned table despite error
              various functions calculationg partition_id and subpart_id didn't expect
                  an error returned from item->val_int().  Error checks added.
      afcc4e58
  2. 04 Oct, 2008 2 commits
  3. 03 Oct, 2008 6 commits
    • Chad MILLER's avatar
      Merge fix for Bug#35754. · 24ab1b14
      Chad MILLER authored
      24ab1b14
    • Chad MILLER's avatar
      Merge fix for Bug#35754. · 4159b470
      Chad MILLER authored
      4159b470
    • Andrei Elkin's avatar
      merge with the main 5.1-bugteam · 0e2681e9
      Andrei Elkin authored
      0e2681e9
    • Davi Arnaut's avatar
      Bug#37481: status.test fails randomly · 5fcb4607
      Davi Arnaut authored
      The problem was that the test was trying to obtain a lock on
      a table in one connection without ensuring that a insert which
      was executed in another connection had released the lock on the
      same table.
      
      The solution is to add a dummy select query after the insert to
      ensure that the table is unlocked and closed by the time it tries
      to lock it again. This is enough to prevent test failures described
      in the bug report. As an extra safety measure, concurrent inserts
      are disabled.
      
      Remove comments that calculated the Table_locks_immediate. This
      value is not tested anymore and it's calculation did not reflect
      the actual value.
      
      mysql-test/r/status.result:
        Update test case result.
      mysql-test/t/status.test:
        Issue a dummy select to ensure that tables are unlocked after a
        insert and disable concurrent inserts as a extra-safety.
      5fcb4607
    • Davi Arnaut's avatar
      Wrap function used for debugging with the DBUG_OFF condition. · 1c453040
      Davi Arnaut authored
      Function uses a variable only available under debug mode.
      1c453040
    • Andrei Elkin's avatar
      Bug #28786 binlog_killed.test fails: 'reset master' does not reset binlogging · 66aa0210
      Andrei Elkin authored
      The test failed originally -- did not reset binlogging - for the reason
      identified by bug@15580.
      However it never can be run on the embedded platfrom for yet another cause -
      the embedded can not KILL query.
            
      Comments added to the test particularly relating `reset master'
      to the mentioned bug.
      
      mysql-test/suite/binlog/t/binlog_killed.test:
        adding commets explaining why the test has to have include/not_embedded.inc guard.
      66aa0210
  4. 02 Oct, 2008 15 commits
  5. 01 Oct, 2008 16 commits
    • Matthias Leich's avatar
      Fix for Bug#37744 Expected result of "<engine>_storedproc" test · f9427665
      Matthias Leich authored
                        is inconsistent
      + several improvements
      Details:
      - The subtest with assignment of floating point numbers to
        DECIMAL parameters in functions and procedures checks
        now that the final DECIMAL value is the same as if we assign
        the floating point numbers to columns, user variables etc.
        = The impact of math libs or truncation must be the same.
      - Remove storage engine variants of this test because the
        stored procedure properties tested do not depend on
        the storage engine.
        Use the fastest storage engine (MEMORY) for any tables
        needed.
      - reset global sort_buffer_size to startup value
      - Partially improved formatting.
      f9427665
    • Ingo Struewing's avatar
      merge · 7abc96fb
      Ingo Struewing authored
      7abc96fb
    • Georgi Kodinov's avatar
      8bea469c
    • Georgi Kodinov's avatar
      Bug#37943: Reproducible mysqld crash/sigsegv in sel_trees_can_be_ored · 73be5f95
      Georgi Kodinov authored
                  
      When analyzing the possible index use cases the server was re-using an internal structure.
      This is wrong, as this internal structure gets updated during the analysis.
      Fixed by making a copy of the internal structure for every place it needs to be used.
      Also stopped the generation of empty SEL_TREE structures that unnecessary 
      complicate the analysis.
      
      mysql-test/r/index_merge.result:
        Bug#37943: test case
      mysql-test/t/index_merge.test:
        Bug#37943: test case
      sql/opt_range.cc:
        Bug#37943: 
         - Make copy constructors for SEL_TREE and sub-structures and use them when OR-ing trees.
         - don't generate empty SEL_TREEs. Return NULL instead.
      73be5f95
    • Ingo Struewing's avatar
      merge · 4737a081
      Ingo Struewing authored
      4737a081
    • Sven Sandberg's avatar
      BUG#38269: pushbuild gives valgrind error in ha_statistic_increment for rpl_temporary · da4cc11b
      Sven Sandberg authored
      Re-enabling failing test case because server logs were lost in pushbuild, so we need to run it again.
      
      
      mysql-test/suite/rpl/t/disabled.def:
        Re-enabling failing test case because server logs were lost in pushbuild, so we need to run it again.
      da4cc11b
    • Georgi Kodinov's avatar
      5721d2d9
    • Georgi Kodinov's avatar
    • Ingo Struewing's avatar
      merge · 8e488336
      Ingo Struewing authored
      8e488336
    • Ingo Struewing's avatar
      Bug#37958 - test main.plugin crash on Mac OS X when selecting from EXAMPLE engine. · a85379af
      Ingo Struewing authored
      This patch contains fixes for two problems:
      
      1. As originally reported, the server crashed on Mac OS X when trying to access
         an EXAMPLE table after the EXAMPLE plugin was installed.
      
         It turned out that the dynamically loaded EXAMPLE plugin called the
         function hash_earch() from a Mac OS X system library, instead of
         hash_earch() from MySQL's mysys library. Makefile.am in storage/example
         does not include libmysys. So the Mac OS X linker arranged the hash_search()
         function to be linked to the system library when the shared object is
         loaded.
      
         One possible solution would be to include libmysys into the linkage of
         dynamic plugins. But then we must have a libmysys.so, which must be
         used by the server too. This could have a minimal performance impact,
         but foremost the change seems to bee too risky at the current state of
         MySQL 5.1.
      
         The selected solution is to rename MySQL's hash_search() to my_hash_search()
         like it has been done before with hash_insert() and hash_reset().
      
         Since this is the third time, we need to rename a hash_*() function,
         I did renamed all hash_*() functions to my_hash_*().
      
         To avoid changing a zillion calls to these functions, and announcing
         this to hundreds of developers, I added defines that map the old names
         to the new names.
      
         This change is in hash.h and hash.c.
      
      2. The other problem was improper implementation of the handlerton-to-plugin
         mapping. We use a fixed-size array to hold a plugin reference for each
         handlerton. On every install of a handler plugin, we allocated a new slot
         of the array. On uninstall we did not free it. After some uninstall/install
         cycles the array overflowed. We did not check for overflow.
      
         One fix is to check for overflow to stop the crashes.
      
         Another fix is to free the array slot at uninstall and search for a free slot
         at plugin install.
      
         This change is in handler.cc.
      
      
      
      include/hash.h:
        Bug#37958 - test main.plugin crash on Mac OS X when selecting from EXAMPLE engine.
        Renamed hash_*() functions to my_hash_*().
        Added defines that map old names to new names.
      mysys/hash.c:
        Bug#37958 - test main.plugin crash on Mac OS X when selecting from EXAMPLE engine.
        Renamed hash_*() functions to my_hash_*().
      sql/handler.cc:
        Bug#37958 - test main.plugin crash on Mac OS X when selecting from EXAMPLE engine.
        Protect against a failing ha_initialize_handlerton() in ha_finalize_handlerton().
        Free hton2plugin slot on uninstall of a handler plugin.
        Reuse freed slost of the hton2plugin array.
        Protect against array overrun.
      a85379af
    • Georgi Kodinov's avatar
      38d8ea5c
    • Georgi Kodinov's avatar
      merged 5.0-bugteam -> 5.1-bugteam · f0124682
      Georgi Kodinov authored
      f0124682
    • Georgi Kodinov's avatar
      merged 5.1-5.1.29-rc -> 5.1-bugteam · adf60749
      Georgi Kodinov authored
      adf60749
    • Georgi Kodinov's avatar
      fixed a wrong directory in distinct.test · fbb331c2
      Georgi Kodinov authored
      fbb331c2
    • Georgi Kodinov's avatar
      merged 5.0-main -> 5.0-bugteam · e2a3b58c
      Georgi Kodinov authored
      e2a3b58c
    • Georgi Kodinov's avatar
      merged 5.0-5.1.29-rc into 5.0-bugteam · 9fa56ec7
      Georgi Kodinov authored
      9fa56ec7