1. 29 Mar, 2007 3 commits
    • unknown's avatar
      Merge pilot.blaudden:/home/msvensson/mysql/bug25482/my41-bug25482-alt2 · b1e3382c
      unknown authored
      into  pilot.blaudden:/home/msvensson/mysql/mysql-4.1-maint
      
      b1e3382c
    • unknown's avatar
      Merge bk-internal:/home/bk/mysql-4.1-maint · 4ab858d2
      unknown authored
      into  pilot.blaudden:/home/msvensson/mysql/mysql-4.1-maint
      
      4ab858d2
    • unknown's avatar
      Bug#25482 GRANT statements are not replicated if you use "replicate-ignore-table" · 66036157
      unknown authored
       - GRANT and REVOKE statments didn't have the "updating" flag set and
         thus statements with a table specified would not replicate if
         slave filtering rules where turned on.
         For example "GRANT ... ON test.t1 TO ..." would not replicate.
      
      
      mysql-test/r/rpl_ignore_table.result:
        Add test results
      mysql-test/t/rpl_ignore_table.test:
        Add tests
      sql/sql_yacc.yy:
        Pass option TL_OPTION_UPDATING to 'add_table_to_list' when parsing a
        GRANT or REVOKE and a table specifier is found. This will set the
        property "updating" on the table and thus the slave filtering rules will 
        be applied.
        
        Without setting updating the statement will be not
        replicated - since "it's not updating anything" - an optimization
        to quickly skip SELECT's and similar.
      66036157
  2. 28 Mar, 2007 1 commit
    • unknown's avatar
      Bug #26642: create index corrupts table definition in .frm · c37bb974
      unknown authored
      Thanks to Martin Friebe for finding and submitting a fix for this bug!
      
      A table with maximum number of key segments and maximum length key name
      would have a corrupted .frm file, due to an incorrect calculation of the
      complete key length.  Now the key length is computed correctly (I hope) :-)
      
      MyISAM would reject a table with the maximum number of keys and the maximum
      number of key segments in all keys.  It would allow one less than this total
      maximum.  Now MyISAM accepts a table defined with the maximum.  (This is a
      very minor issue.)
      
      
      myisam/mi_open.c:
        change >= to > in a comparison (i.e., error only if key_parts_in_table
        really is greater than MAX_KEY * MAX_KEY_SEG)
      mysql-test/r/create.result:
        Add test results for bug #26642 (create index corrupts table definition in .frm)
      mysql-test/t/create.test:
        Add test case for bug #26642 (create index corrupts table definition in .frm)
      sql/table.cc:
        In create_frm(), fix formula for key_length; it was too small by (keys * 2) bytes
      c37bb974
  3. 26 Mar, 2007 1 commit
  4. 21 Mar, 2007 1 commit
  5. 20 Mar, 2007 2 commits
  6. 15 Mar, 2007 2 commits
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-4.1 · 3696e9d8
      unknown authored
      into  mockturtle.local:/home/dlenev/src/mysql-4.1-merge
      
      3696e9d8
    • unknown's avatar
      Fix for bug #25966 "2MB per second endless memory consumption after LOCK · 3dae7ac8
      unknown authored
      TABLE ... WRITE".
      
      CPU hogging occured when connection which had to wait for table lock was
      serviced by thread which previously serviced connection that was killed
      (note that connections can reuse threads if thread cache is enabled).
      One possible scenario which exposed this problem was when thread which
      provided binlog dump to replication slave was implicitly/automatically
      killed when the same slave reconnected and started pulling data through
      different thread/connection.
      In 5.* versions memory hogging was added to CPU hogging. Moreover in
      those versions the problem also occured when one killed particular query
      in connection (using KILL QUERY) and later this connection had to wait for
      some table lock.
      
      This problem was caused by the fact that thread-specific mysys_var::abort
      variable, which indicates that waiting operations on mysys layer should
      be aborted (this includes waiting for table locks), was set by kill
      operation but was never reset back. So this value was "inherited" by the
      following statements or even other connections (which reused the same
      physical thread). Such discrepancy between this variable and THD::killed
      flag broke logic on SQL-layer and caused CPU and memory hogging.
      
      This patch tries to fix this problem by properly resetting this member.
      
      There is no test-case associated with this patch since it is hard to test
      for memory/CPU hogging conditions in our test-suite.
      
      
      sql/mysqld.cc:
        We should not forget to reset THD::mysys_var::abort after kill operation
        if we are going to use thread to which this operation was applied for
        handling of other connections.
      3dae7ac8
  7. 14 Mar, 2007 5 commits
  8. 13 Mar, 2007 2 commits
    • unknown's avatar
      BUG#26881 - Large MERGE tables report incorrect specification when no · 4d7f2c22
      unknown authored
                  differences in tables
      Certain merge tables were wrongly reported as having incorrect definition:
      - Some fields that are 1 byte long (e.g. TINYINT, CHAR(1)), might
        be internally casted (in certain cases) to a different type on a
        storage engine layer. (affects 4.1 and up)
      - If tables in a merge (and a MERGE table itself) had short VARCHAR column (less
        than 4 bytes) and at least one (but not all) tables were ALTER'ed (even to an
        identical table: ALTER TABLE xxx ENGINE=yyy), table definitions went ouf of
        sync. (affects 4.1 only)
      
      This is fixed by relaxing a check for underlying conformance and setting
      field type to FIELD_TYPE_STRING in case varchar is shorter than 4
      when a table is created.
      
      
      myisam/mi_create.c:
        Added a comment.
      mysql-test/r/merge.result:
        A test case for bug#26881.
      mysql-test/t/merge.test:
        A test case for bug#26881.
      sql/ha_myisam.cc:
        Relaxed some checks performed by check_definition():
        As comparing of fulltext keys (and key segments) is not yet implemented,
        only return an error in case one of keys is fulltext and other is not.
        Otherwise, if both keys are fulltext, accept them as is.
        
        As comparing of spatial keys (and key segments) is not yet implemented,
        only return an error in case one of keys is spatial and other is not.
        Otherwise, if both keys are spatial, accept them as is.
        
        A workaround to handle situation when field is casted from FIELD_SKIP_ZERO
        to FIELD_NORMAL. This could happen only in case field length is 1 and row
        format is fixed.
      sql/sql_parse.cc:
        When a table that has varchar field shorter than 4 is created, field type is
        set to FIELD_TYPE_VAR_STRING. Later, when a table is modified using alter
        table, field type is changed to FIELD_TYPE_STRING (see Field_string::type).
        That means HA_OPTION_PACK_RECORD flag might be lost and thus null_bit might
        be shifted by alter table, in other words alter table doesn't create 100%
        equal table definition.
        
        This is usually not a problem, since when a table is created/altered,
        definition on a storage engine layer is based on one that is passed from
        sql layer. But it is a problem for merge engine - null_bit is shifted when
        a table (merge or underlying) is altered.
        
        Set field type to FIELD_TYPE_STRING in case FIELD_TYPE_VAR_STRING is shorter
        than 4 when a table is created as it is done in Field::type.
      4d7f2c22
    • unknown's avatar
      Merge mysql.com:/home/svoj/devel/bk/mysql-4.1 · e90872a4
      unknown authored
      into  mysql.com:/home/svoj/devel/mysql/BUG26881/mysql-4.1-engines
      
      e90872a4
  9. 12 Mar, 2007 4 commits
    • unknown's avatar
      configure.in: · 91ea1199
      unknown authored
        Restored accidently removed line to check for zlib
      
      
      configure.in:
        Restored accidently removed line to check for zlib
      91ea1199
    • unknown's avatar
      mysys_ia64.dsp, mysys.vcproj: · c9f146bc
      unknown authored
        Removed references to unused files
      
      
      VC++Files/mysys/mysys.vcproj:
        Removed unused files
      VC++Files/mysys/mysys_ia64.dsp:
        Removed unused files
      c9f146bc
    • unknown's avatar
      Merge mysql.com:/home/kent/bk/tmp/mysql-4.0 · 84ea1a3a
      unknown authored
      into  mysql.com:/home/kent/bk/tmp/mysql-4.1-build
      
      
      include/Makefile.am:
        Auto merged
      84ea1a3a
    • unknown's avatar
      Makefile.am, configure.in, mysys.dsp: · 96695de3
      unknown authored
        Removed unused files
      .del-my_winsem.c:
        Delete: mysys/my_winsem.c
      .del-my_semaphore.c:
        Delete: mysys/my_semaphore.c
      .del-my_semaphore.h:
        Delete: include/my_semaphore.h
      
      
      BitKeeper/deleted/.del-my_semaphore.c:
        Delete: mysys/my_semaphore.c
      BitKeeper/deleted/.del-my_semaphore.h:
        Delete: include/my_semaphore.h
      BitKeeper/deleted/.del-my_winsem.c:
        Delete: mysys/my_winsem.c
      VC++Files/mysys/mysys.dsp:
        Removed unused files
      configure.in:
        Removed unused files
      include/Makefile.am:
        Removed unused files
      mysys/Makefile.am:
        Removed unused files
      96695de3
  10. 10 Mar, 2007 1 commit
  11. 09 Mar, 2007 1 commit
  12. 08 Mar, 2007 5 commits
    • unknown's avatar
      Bug#25673 - spatial index corruption, error 126 incorrect key file for table · 133c32b3
      unknown authored
      After backport fix.
      Added forgotten DBUG_RETURNs, which was detected in 5.1 only.
      
      133c32b3
    • unknown's avatar
      Merge bk@192.168.21.1:mysql-4.1 · 421ebc9c
      unknown authored
      into  mysql.com:/home/hf/work/mrg/mysql-4.1-opt
      
      421ebc9c
    • unknown's avatar
      Merge bk-internal:/home/bk/mysql-4.1-maint · ff39c451
      unknown authored
      into  pilot.blaudden:/home/msvensson/mysql/mysql-4.1-maint
      
      ff39c451
    • unknown's avatar
      Bug#25673 - spatial index corruption, error 126 · 717b23af
      unknown authored
                incorrect key file for table
      
      In certain cases it could happen that deleting a row could
      corrupt an RTREE index.
      
      According to Guttman's algorithm, page underflow is handled
      by storing the page in a list for later re-insertion. The
      keys from the stored pages have to be inserted into the
      remaining pages of the same level of the tree. Hence the
      level number is stored in the re-insertion list together
      with the page.
      
      In the MySQL RTree implementation the level counts from zero
      at the root page, increasing numbers for levels down the tree.
      
      If during re-insertion of the keys the tree height grows, all
      level numbers become invalid. The remaining keys will be
      inserted at the wrong level.
      
      The fix is to increment the level numbers stored in the
      reinsert list after a split of the root block during reinsertion.
      
      
      myisam/rt_index.c:
        Bug#25673 - spatial index corruption, error 126
                    incorrect key file for table
        Added a loop in rtree_delete() to increment the level numbers
        stored in the reinsert list after a split of the root block
        during reinsertion.
        Added comments and DBUG statements.
      myisam/rt_key.c:
        Bug#25673 - spatial index corruption, error 126
                    incorrect key file for table
        Added DBUG statements.
      myisam/rt_split.c:
        Bug#25673 - spatial index corruption, error 126
                    incorrect key file for table
        Added DBUG statements.
      mysql-test/r/gis-rtree.result:
        Bug#25673 - spatial index corruption, error 126
                    incorrect key file for table
        Added the test result.
      mysql-test/t/gis-rtree.test:
        Bug#25673 - spatial index corruption, error 126
                    incorrect key file for table
        Added a test.
      717b23af
    • unknown's avatar
      Merge quadxeon.mysql.com:/benchmarks/ext3/TOSAVE/tsmith/bk/maint/mar07/stage2/40 · 45f97715
      unknown authored
      into  quadxeon.mysql.com:/benchmarks/ext3/TOSAVE/tsmith/bk/maint/mar07/stage2/41
      
      45f97715
  13. 07 Mar, 2007 5 commits
  14. 05 Mar, 2007 4 commits
    • unknown's avatar
      Merge rkalimullin@bk-internal.mysql.com:/home/bk/mysql-4.1-maint · 2d5b22c9
      unknown authored
      into  mysql.com:/home/ram/work/b26038/b26038.4.1
      
      2d5b22c9
    • unknown's avatar
      Bug#26464 - insert delayed + update + merge = corruption · 730ace19
      unknown authored
      Using INSERT DELAYED on MERGE tables could lead to table
      corruptions.
      
      The manual lists a couple of storage engines, which can be
      used with INSERT DELAYED. MERGE is not in this list.
      
      The attempt to try it anyway has not been rejected yet.
      This bug was not detected earlier as it can work under
      special circumstances. Most notable is low concurrency.
      
      To be safe, this patch rejects any attempt to use INSERT
      DELAYED on MERGE tables.
      
      
      mysql-test/r/merge.result:
        Bug#26464 - insert delayed + update + merge = corruption
        Added test result.
      mysql-test/t/merge.test:
        Bug#26464 - insert delayed + update + merge = corruption
        Added test.
      sql/ha_myisammrg.h:
        Bug#26464 - insert delayed + update + merge = corruption
        Removed HA_CAN_INSERT_DELAYED flag from table_flags().
        The insert delayed thread upgrades the lock from the first
        entry in MYSQL_LOCK::locks only. Hence it is incapable to
        handle MERGE tables, which have as many entries in this
        array as they have MyISAM sub-tables.
      730ace19
    • unknown's avatar
      Bug #26792 Add DBX debugger support to mysql-test-run.pl · 96509328
      unknown authored
       - Add --debugger=dbx
       - Fix --debugger=devenv, --debugger=DevEnv and --debugger=/path/devenv
      
      
      mysql-test/mysql-test-run.pl:
        Add support for --debugger=dbx to mysql-test-run.pl
        Fix case senitive match for vc, vcexpress or deven
        Make it possible to use full path to debugger for
        --debugger=/path/vcexpress
      96509328
    • unknown's avatar
      Merge rkalimullin@bk-internal.mysql.com:/home/bk/mysql-4.1-maint · 5d9013a4
      unknown authored
      into  mysql.com:/home/ram/work/b23616/b23616.4.1
      
      5d9013a4
  15. 03 Mar, 2007 1 commit
    • unknown's avatar
      Bug#25126: Wrongly resolved field leads to a crash. · d524534d
      unknown authored
      When the ORDER BY clause gets fixed it's allowed to search in the current
      item_list in order to find aliased fields and expressions. This is ok for a
      SELECT but wrong for an UPDATE statement. If the ORDER BY clause will
      contain a non-existing field which is mentioned in the UPDATE set list
      then the server will crash due to using of non-existing (0x0) field.
      
      When an Item_field is getting fixed it's allowed to search item list for
      aliased expressions and fields only for selects.
      
      
      sql/sql_base.cc:
        Bug#25126: Wrongly resolved field leads to a crash.
        When an Item_field is getting fixed it's allowed to search item list for
        aliased expressions and fields only for selects.
      sql/sql_select.cc:
        Bug#25126: Wrongly resolved field leads to a crash.
        When an Item_field is getting fixed it's allowed to search item list for
        aliased expressions and fields only for selects.
      mysql-test/r/update.result:
        Added a test case for bug#25126: Wrongly resolved field leads to a crash.
      mysql-test/t/update.test:
        Added a test case for bug#25126: Wrongly resolved field leads to a crash.
      d524534d
  16. 02 Mar, 2007 2 commits