1. 24 Jun, 2007 3 commits
    • unknown's avatar
      Merge olga.mysql.com:/home/igor/mysql-5.0-opt · 683a1119
      unknown authored
      into  olga.mysql.com:/home/igor/dev-opt/mysql-5.0-opt-bug25602
      
      
      sql/sql_select.cc:
        Auto merged
      683a1119
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0 · 98646c96
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.0-opt
      
      
      sql/log_event.cc:
        Auto merged
      98646c96
    • unknown's avatar
      Fixed bug #25602. A query with DISTINCT in the select list to which · 09e2b603
      unknown authored
      the loose scan optimization for grouping queries was applied returned 
      a wrong result set when the query was used with the SQL_BIG_RESULT
      option.
      
      The SQL_BIG_RESULT option forces to use sorting algorithm for grouping
      queries instead of employing a suitable index. The current loose scan
      optimization is applied only for one table queries when the suitable
      index is covering. It does not make sense to use sort algorithm in this
      case. However the create_sort_index function does not take into account
      the possible choice of the loose scan to implement the DISTINCT operator
      which makes sorting unnecessary. Moreover the current implementation of
      the loose scan for queries with distinct assumes that sorting will
      never happen. Thus in this case create_sort_index should not call
      the function filesort.
      
      
      mysql-test/r/group_min_max.result:
        Added a test case for bug #25602.
      mysql-test/t/group_min_max.test:
        Added a test case for bug #25602.
      09e2b603
  2. 23 Jun, 2007 3 commits
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/4.1-opt · 6688a54f
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.0-opt
      
      6688a54f
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt-29095 · e45576ac
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.0-opt
      
      e45576ac
    • unknown's avatar
      Fixed bug #29095. · e7778b65
      unknown authored
      INSERT into table from SELECT from the same table
      with ORDER BY and LIMIT was inserting other data
      than sole SELECT ... ORDER BY ... LIMIT returns.
      
      One part of the patch for bug #9676 improperly pushed
      LIMIT to temporary table in the presence of the ORDER BY
      clause.
      That part has been removed.
      
      
      sql/sql_select.cc:
        Fixed bug #29095.
        One part of the patch for bug #9676 improperly pushed
        LIMIT to temporary table in the presence of the ORDER BY
        clause.
        That part has been removed.
      mysql-test/t/insert_select.test:
        Expanded the test case for bug #9676.
        Created a test case for bug #29095.
      mysql-test/r/insert_select.result:
        Expanded the test case for bug #9676.
        Created a test case for bug #29095.
      e7778b65
  3. 22 Jun, 2007 6 commits
    • unknown's avatar
      Merge gkodinov@bk-internal.mysql.com:/home/bk/mysql-5.0-opt · 9f368072
      unknown authored
      into  magare.gmz:/home/kgeorge/mysql/autopush/B28400-5.0-opt
      
      9f368072
    • unknown's avatar
      Bug #27383: Crash in test "mysql_client_test" · 0200ffc7
      unknown authored
      The C optimizer may decide that data access operations
      through pointer of different type are not related to 
      the original data (strict aliasing).
      This is what happens in fetch_long_with_conversion(),
      when called as part of mysql_stmt_fetch() : it tries 
      to check for truncation errors by first storing float
      (and other types of data) into a char * buffer and then 
      accesses them through a float pointer.
      This is done to prevent the effects of excess precision
      when using FPU registers.
      However the doublestore() macro converts a double pointer
      to an union pointer. This violates the strict aliasing rule.
      Fixed by making the intermediary variables volatile (
      to not re-introduce the excess precision bug) and using
      the intermediary value instead of the char * buffer.
      Note that there can be loss of precision for both signed
      and unsigned 64 bit integers converted to double and back,
      so the check must stay there (even for compatibility 
      reasons).
      Based on the excellent analysis in bug 28400.
      
      
      libmysql/libmysql.c:
        Bug #27383: avoid pointer aliasing problems while 
        not re-violating the Intel FPU gcc bug.
      0200ffc7
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.0-maint · 3300b8eb
      unknown authored
      into  maint1.mysql.com:/data/localhome/tsmith/bk/maint/50
      
      3300b8eb
    • unknown's avatar
      Merge bk@192.168.21.1:mysql-5.0-opt · 0adc47a6
      unknown authored
      into  mysql.com:/home/hf/work/28839/my50-28839
      
      0adc47a6
    • unknown's avatar
      rpl_skip_error.test fixed · d58aeec1
      unknown authored
      
      mysql-test/r/rpl_skip_error.result:
        test result fixed
      mysql-test/t/rpl_skip_error.test:
        inconsistent column results hidden
      d58aeec1
    • unknown's avatar
      Bug #29138 'kill' fails in pushbuild · 86ae7940
      unknown authored
      The reason the "reap;" succeeds unexpectedly is because the query was completing(almost always) and the network buffer was big enough to store the query result (sometimes) on Windows, meaning the response was completely sent before the server thread could be killed.
      
      Therefore we use a much longer running query that doesn't have a chance to fully complete before the reap happens, testing the kill properly.
      
      
      mysql-test/r/kill.result:
        We use a much longer running query that doesn't have a chance to fully complete before the reap happens.
      mysql-test/t/kill.test:
        We use a much longer running query that doesn't have a chance to fully complete before the reap happens.
      86ae7940
  4. 21 Jun, 2007 16 commits
    • unknown's avatar
      Merge olga.mysql.com:/home/igor/mysql-5.0-opt · c0cc2fee
      unknown authored
      into  olga.mysql.com:/home/igor/dev-opt/mysql-5.0-opt-bug29104
      
      
      sql/item.cc:
        Auto merged
      c0cc2fee
    • unknown's avatar
      Merge maint1.mysql.com:/data/localhome/tsmith/bk/maint/41 · 839c07db
      unknown authored
      into  maint1.mysql.com:/data/localhome/tsmith/bk/maint/50
      
      839c07db
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.0-rpl · 6367bd2a
      unknown authored
      into  maint1.mysql.com:/data/localhome/tsmith/bk/maint/50
      
      
      mysql-test/t/innodb.test:
        Auto merged
      mysql-test/r/innodb.result:
        Manual merge
      6367bd2a
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-4.1-rpl · 42247162
      unknown authored
      into  maint1.mysql.com:/data/localhome/tsmith/bk/maint/41
      
      42247162
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.0-maint · d3662296
      unknown authored
      into  amd64.(none):/src/bug27029/my50-bug27029
      
      
      myisam/sort.c:
        Auto merged
      d3662296
    • unknown's avatar
      Bug#27029 alter table ... enable keys crashes mysqld on large table · 8d27f335
      unknown authored
      - When creating an index for the sort, the number of rows plus 1 is used 
      to allocate a buffer.  In this test case, the number of rows 4294967295 
      is the max value of an unsigned integer, so when 1 was added to it, a 
      buffer of size 0 was allocated causing the crash.
      - Create new test suite for this bug's test suite as per QA.
      
      
      myisam/sort.c:
        Bug#27029 alter table ... enable keys crashes mysqld on large table
        - Check to make sure the value of records is < UINT_MAX32 to avoid a 
        false positive on the remaining condition.
      mysql-test/suite/large_tests/README.TXT:
        Bug#27029 alter table ... enable keys crashes mysqld on large table
        - New testsuite for large tests.
        - Added mtr hint for potential testers.
      mysql-test/suite/large_tests/r/alter_table.result:
        Bug#27029 alter table ... enable keys crashes mysqld on large table
        - New testsuite for large tests.
        - Added results for bug.
      mysql-test/suite/large_tests/t/alter_table.test:
        Bug#27029 alter table ... enable keys crashes mysqld on large table
        - New testsuite for large tests.
        - Added test for bug.
      8d27f335
    • unknown's avatar
      Merge maint1.mysql.com:/data/localhome/tsmith/bk/50 · 6c70aa11
      unknown authored
      into  maint1.mysql.com:/data/localhome/tsmith/bk/maint/50
      
      
      configure.in:
        Auto merged
      6c70aa11
    • unknown's avatar
      Merge mysql.com:/nfsdisk1/lars/bk/mysql-5.0 · 9dfd4335
      unknown authored
      into  mysql.com:/nfsdisk1/lars/bk/mysql-5.0-rpl
      
      9dfd4335
    • unknown's avatar
      Test fix · aacbe21c
      unknown authored
      aacbe21c
    • unknown's avatar
      66e76447
    • unknown's avatar
      Merge pilot.(none):/data/msvensson/mysql/bug28769/my50-bug28769 · 863e3da3
      unknown authored
      into  pilot.(none):/data/msvensson/mysql/mysql-5.0-maint
      
      863e3da3
    • unknown's avatar
      Merge pilot.(none):/data/msvensson/mysql/bug28742/my50-bug28742 · bd099cd8
      unknown authored
      into  pilot.(none):/data/msvensson/mysql/mysql-5.0-maint
      
      
      mysql-test/mysql-test-run.pl:
        Auto merged
      bd099cd8
    • unknown's avatar
      Merge bk@192.168.21.1:mysql-5.0-opt · 7d95000e
      unknown authored
      into  mysql.com:/home/hf/work/28839/my50-28839
      
      7d95000e
    • unknown's avatar
      rpl_skip_error test fixed · 0110a105
      unknown authored
      
      mysql-test/r/rpl_skip_error.result:
        result fixed
      mysql-test/t/rpl_skip_error.test:
        master port dependency eliminated
      0110a105
    • unknown's avatar
      Merge bk@192.168.21.1:mysql-5.0-opt · 8fa4b2fc
      unknown authored
      into  mysql.com:/home/hf/work/28839/my50-28839
      
      
      sql/log_event.cc:
        Auto merged
      8fa4b2fc
    • unknown's avatar
      Bug#24924: shared-memory-base-name that is too long causes buffer overflow · 26e9aba4
      unknown authored
      long shared-memory-base-names could overflow a static internal buffer
      and thus crash mysqld and various clients.  change both to dynamic
      buffers, show everything but overflowing those buffers still works.
      
      The test case for this would pretty much amount to
      mysqld --shared-memory-base-name=HeyMrBaseNameXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX --shared-memory=1 &
      mysqladmin --no-defaults --shared-memory-base-name=HeyMrBaseNameXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX shutdown
      
      Unfortunately, we can't just use an .opt file for the
      server. The .opt file is used at start-up, before any
      include in the actual test can tell mysqltest to skip
      this one on non-Windows. As a result, such a test would
      break on unices.
      
      Fixing mysql-test-run.pl to export full path for master
      and slave would enable us to start a server from within
      the test which is ugly and, what's more, doesn't work as
      the server blocks (mysqltest offers no fire-and-forget
      fork-and-exec), and mysqladmin never gets run.
      
      Making the test rpl_windows_shm or some such so we can
      is beyond ugly. As is introducing another file-name based
      special case (run "win*.test" only when on Windows). As is
      (yuck) coding half the test into mtr (as in, having it
      hand out a customized environment conductive to the shm-
      thing on Win only).
      
      Situation is exacerbated by the fact that .sh is not
      necessary run as expected on Win.
      
      In short, it's just not worth it. No test-case until we
      have a new-and-improved test framework.
      
      
      sql-common/client.c:
        Bug#24924: shared-memory-base-name that is too long causes buffer overflow
        
        compose shared memory name in dynamic rather than static buffer to prevent
        overflows (clients)
      sql/mysqld.cc:
        Bug#24924: shared-memory-base-name that is too long causes buffer overflow
        
        compose shared memory name in dynamic rather than static buffer to prevent
        overflows (server)
      26e9aba4
  5. 20 Jun, 2007 12 commits
    • unknown's avatar
      Fixed bug #28293. · 86b04e8b
      unknown authored
      Occasionally mysqlbinlog --hexdump failed with error:
        ERROR 1064 (42000) at line ...: You have an error in your
        SQL syntax; check the manual that corresponds to your MySQL
        server version for the right syntax to use near
        'Query thread_id=... exec_time=... error_code=...
      
      When the length of hexadecimal dump of binlog header was
      divisible by 16, commentary sign '#' after header was lost.
      The Log_event::print_header function has been modified to always
      finish hexadecimal binlog header with "\n# ".
      
      
      sql/log_event.cc:
        Fixed bug #28293.
        The Log_event::print_header function has been modified to always
        finish hexadecimal binlog header with "\n# ".
      mysql-test/r/mysqlbinlog.result:
        Updated test case for bug #28293.
      mysql-test/t/mysqlbinlog.test:
        Updated test case for bug #28293.
      86b04e8b
    • unknown's avatar
      Fixed bug #29104: assertion abort for grouping queries using views. · 45e5f9f3
      unknown authored
      The abort happened when a query contained a conjunctive predicate
      of the form 'view column = constant' in the WHERE condition and 
      the grouping list also contained a reference to a view column yet
      a different one.
      
      Removed the failing assertion as invalid in a general case.
      
      Also fixed a bug that prevented applying some optimization for grouping
      queries using views. If the WHERE condition of such a query contains
      a conjunctive condition of the form 'view column = constant' and
      this view column is used in the grouping list then grouping by this
      column can be eliminated. The bug blocked performing this elimination.
      
      
      mysql-test/r/view.result:
        Added a test case for bug #29104.
      mysql-test/t/view.test:
        Added a test case for bug #29104.
      sql/item.cc:
        Fixed bug #29104: assertion abort for grouping queries using views.
        The abort happened when a query contained a conjunctive predicate
        of the form 'view column = constant' in the WHERE condition and 
        the grouping list also contained a reference to a view column yet
        a different one.
        
        Removed the failing assertion as invalid in a general case.
        
        Also fixed a bug that prevented applying some optimization for grouping
        queries using views. If the WHERE condition of such a query contains
        a conjunctive condition of the form 'view column = constant' and
        this view column is used in the grouping list then grouping by this
        column can be eliminated. The bug blocked performing this elimination.
        This bug was in the function Item_field::eq while the failing
        assertion was in the function Item_direct_view_ref::eq.
      45e5f9f3
    • unknown's avatar
      Merge kindahl-laptop.dnsalias.net:/home/bkroot/mysql-5.0-rpl · 0f3bc4cd
      unknown authored
      into  kindahl-laptop.dnsalias.net:/home/bk/b29030-mysql-5.0-rpl
      
      0f3bc4cd
    • unknown's avatar
      BUG#29030 (DROP USER command that errors still gets written to binary log · 15375044
      unknown authored
      and replicated):
      
      A DROP USER statement with a non-existing user was correctly written to
      the binary log (there might be users that were removed, but not all),
      but the error code was not set, which caused the slave to stop with an
      error.
      
      The error reporting code was moved to before the statement was logged
      to ensure that the error information for the thread was correctly set
      up. This works since my_error() will set the fields net.last_errno and
      net.last_error for the thread that is reporting the error, and this
      will then be picked up when the Query_log_event is created and written
      to the binary log.
      
      
      sql/sql_acl.cc:
        Moving error reporting code to ensure that thd->net.last_err{or,no} is
        set and adding debug printout.
      mysql-test/r/rpl_grant.result:
        New BitKeeper file ``mysql-test/r/rpl_grant.result''
      mysql-test/t/rpl_grant.test:
        New BitKeeper file ``mysql-test/t/rpl_grant.test''
      15375044
    • unknown's avatar
      Bug #27583 slave sql fails to read from iocache when slave got stopped at pos==4 · 1b42af2c
      unknown authored
      forgotten merge with 5.0. There can be some bugs waiting for this fix in 5.0 like Bug@29232
      
      
      sql/log_event.cc:
        file name is changed in step with the position.
      1b42af2c
    • unknown's avatar
      Merge gkodinov@bk-internal.mysql.com:/home/bk/mysql-4.1-opt · 37059a7e
      unknown authored
      into  magare.gmz:/home/kgeorge/mysql/autopush/B29116-4.1-opt
      
      37059a7e
    • unknown's avatar
      port of the fix for bug 19116 4.1-opt -> 5.0-opt · 0186f295
      unknown authored
      0186f295
    • unknown's avatar
      Bug#28742 mysql-test-run is very slow on "Stopping All Servers" step · 8c1c0ed1
      unknown authored
       - Improve shutdown algorithm 
       - Wait up to 5 seconds for processes to exit after their port is free
      
      
      mysql-test/lib/mtr_process.pl:
        Improve shutdown algorithm, shutdown the server hard
        if it hasn't responded to "mysqladmin shutdown" and it's port is free.
        Print error to servers error log indicating "hard shutdown"
        Give processes up to 5 seconds to exit after their port is free
      mysql-test/lib/mtr_report.pl:
        Indicate in what file the warning was found
      mysql-test/mysql-test-run.pl:
        Pass path of process error log to 'mtr_check_stop_servers'
      8c1c0ed1
    • unknown's avatar
      Merge magare.gmz:/home/kgeorge/mysql/work/B29116-4.1-opt · 2d9ca378
      unknown authored
      into  magare.gmz:/home/kgeorge/mysql/work/B29116-5.0-opt
      
      
      mysql-test/t/rpl_change_master.test:
        Auto merged
      2d9ca378
    • unknown's avatar
      Merge gleb.loc:/home/uchum/work/bk/5.0-opt-28898 · 77f3c55c
      unknown authored
      into  gleb.loc:/home/uchum/work/bk/5.0-opt
      
      77f3c55c
    • unknown's avatar
      metadata.test, metadata.result: · 39e7beeb
      unknown authored
        Updated test case for bug #28898. Additional cleanup.
      
      
      mysql-test/t/metadata.test:
        Updated test case for bug #28898. Additional cleanup.
      mysql-test/r/metadata.result:
        Updated test case for bug #28898. Additional cleanup.
      39e7beeb
    • unknown's avatar
      Bug #28839 Errors in strict mode silently stop SQL thread if --slave-skip-errors exists. · e3eb0750
      unknown authored
      slave_sql thread calls thd->clear_error() to force error to be ignored,
      though this method didn't clear thd->killed state, what causes
      slave_sql thread to stop.
      
      clear thd->killed state if we ignore an error
      
      
      mysql-test/r/rpl_skip_error.result:
        Bug #28839 Errors in strict mode silently stop SQL thread if --slave-skip-errors exists.
        
        test result
      mysql-test/t/rpl_skip_error.test:
        Bug #28839 Errors in strict mode silently stop SQL thread if --slave-skip-errors exists.
        
        test case
      sql/log_event.cc:
        Bug #28839 Errors in strict mode silently stop SQL thread if --slave-skip-errors exists.
        
        clear thd->killed state if we ignore the error
      e3eb0750