1. 18 Feb, 2008 1 commit
    • unknown's avatar
      Updating result files. · ac68c557
      unknown authored
      
      mysql-test/r/multi_update.result:
        Result change.
      mysql-test/r/rpl_bug33931.result:
        Result change.
      mysql-test/suite/ndb/r/ndb_binlog_format.result:
        Result change.
      ac68c557
  2. 15 Feb, 2008 7 commits
    • unknown's avatar
      Merge dl145h.mysql.com:/data0/mkindahl/mysql-5.0-rpl · 44a2e316
      unknown authored
      into  dl145h.mysql.com:/data0/mkindahl/mysql-5.1-rpl
      
      
      sql/sql_delete.cc:
        Auto merged
      sql/sql_update.cc:
        Auto merged
      BitKeeper/deleted/.del-rpl_variables.test:
        Manual merge.
      mysql-test/r/multi_update.result:
        Manual merge.
      mysql-test/suite/binlog/r/binlog_stm_mix_innodb_myisam.result:
        Manual merge.
      sql/slave.cc:
        Manual merge.
      44a2e316
    • unknown's avatar
      Merge ssandberg@bk-internal.mysql.com:/home/bk/mysql-5.1-new-rpl · 8d97e316
      unknown authored
      into  riska.(none):/home/sven/bktip/5.1-new-rpl
      
      8d97e316
    • unknown's avatar
      Problem: After I moved rpl/t/rpl_variables.test into t/variables.test, · 05d1dc5c
      unknown authored
      t/variables.test fails on embedded server. Moved out that part again, to
      a new test called t/variables-notembedded.test.
      
      
      mysql-test/t/variables-notembedded.test:
        BitKeeper file /home/sven/bk/debug-max/5.1-new-rpl/mysql-test/t/variables-notembedded.test
      mysql-test/r/variables-notembedded.result:
        Add -master.opt file to have a list of variables for slave_skip_errors.
      mysql-test/r/variables.result:
        Variable variables not available in embedded mode to a new test,
        variables-notembedded.test
      mysql-test/t/variables-notembedded-master.opt:
        Add --loose before --slave-skip-errors, so that it works if mtr is run
        with --embedded
      mysql-test/t/variables.test:
        Variable variables not available in embedded mode to a new test,
        variables-notembedded.test. Result file updated accordingly.
      05d1dc5c
    • unknown's avatar
      Merge kindahl-laptop.dnsalias.net:/home/bkroot/mysql-5.1-rpl · df953126
      unknown authored
      into  kindahl-laptop.dnsalias.net:/home/bk/b34458-mysql-5.1-rpl
      
      df953126
    • unknown's avatar
      Bug #34458 Extreneous use of templates in server code · 15267188
      unknown authored
      Patch to remove white-space left over after removing anonymous
      namespace.
      
      
      sql/handler.cc:
        Removing indentation and whitespace.
      15267188
    • unknown's avatar
      Bug #34458 Extreneous use of templates in server code · 06697e88
      unknown authored
      Replacing a template function with a normal static function.
      The template parameter, which previously was the class to
      find a binlogging function in, is now passed as a pointer to
      the actual binlogging function instead.
      
      The patch requires change of indention, but that is submitted
      as a separate patch.
      
      
      sql/handler.cc:
        Replacing template function with a normal static function.
      06697e88
    • unknown's avatar
      Merge abarkov@bk-internal.mysql.com:/home/bk/mysql-5.1-rpl · acb530ee
      unknown authored
      into  mysql.com:/home/bar/mysql-work/mysql-5.1.b27877
      
      acb530ee
  3. 14 Feb, 2008 3 commits
  4. 13 Feb, 2008 2 commits
    • unknown's avatar
      BUG#32991: Races in mysqldump.test (or mysqldump.test fails sporadically) · b1ba8c50
      unknown authored
      This is *not* a fix to the bug. I'm only disabling the failing part of
      mysqldump.test until the bug is fixed. Whoever fixes it, please re-enable
      the test.
      
      
      mysql-test/r/mysqldump.result:
        Disabling part of the test case until BUG#32991 is fixed. Result file
        updated accordingly.
      mysql-test/t/mysqldump.test:
        Disabling this part of the test case until BUG#32991 is fixed.
      b1ba8c50
    • unknown's avatar
      Bug #33931 assertion at write_ignored_events_info_to_relay_log if init_slave_thread() fails · 8b90be89
      unknown authored
      and
      bug#33932  assertion at handle_slave_sql if init_slave_thread() fails
      
      the asserts were caused by 
        bug33931: having thd deleted at time of executing err: code plus
                  a missed initialization;
        bug33932: initialization of slave_is_running member was missed;
      
      fixed with relocating mi members initialization and removing delete thd
      It is safe to do as deletion happens later explicitly in the caller of
      init_slave_thread().
      
      Todo: at merging the test is better to be moved into suite/bugs for 5.x (when x>0).
      
      
      sql/slave.cc:
        adding the bugs simulating code;
        relocating some assignments to satisfy the asserts;
      mysql-test/r/rpl_bug33931.result:
        the new result file
      mysql-test/t/rpl_bug33931-slave.opt:
        option to spark the simulation code
      mysql-test/t/rpl_bug33931.test:
        tests check that slave does not crash as before.
        Slave threads must be in NO running state in the end.
      8b90be89
  5. 12 Feb, 2008 3 commits
    • unknown's avatar
      Moved mysql-test/suite/rpl/rpl_variables.test into · b4302b52
      unknown authored
      mysql-test/t/variables.test, because:
       - mysql-test/suite/rpl/t/rpl_variables.test does not replicate anything,
      so should not be in the rpl suite.
       - mysql-test/t/variables.test is the place for testing variable-related
      problems and features.
       - I will soon commit a patch containing a test case that tests
      replication of variables. It would be good if I could call the test case
      mysql-test/suite/rpl/t/rpl_variables.test. I'm making place for that now.
      
      
      mysql-test/t/variables-master.opt:
        Rename: mysql-test/suite/rpl/t/rpl_variables-master.opt -> mysql-test/t/variables-master.opt
      BitKeeper/deleted/.del-rpl_variables.test:
        Delete: mysql-test/suite/rpl/t/rpl_variables.test
      BitKeeper/deleted/.del-rpl_variables.result:
        Delete: mysql-test/suite/rpl/r/rpl_variables.result
      mysql-test/r/variables.result:
        Moved mysql-test/suite/rpl/t/rpl_variables.test into
        mysql-test/t/variables.test . Result file needs to be updated.
      mysql-test/t/variables.test:
        Moved mysql-test/suite/rpl/t/rpl_variables.test into
        mysql-test/t/variables.test .
      b4302b52
    • unknown's avatar
      Merge mysql1000.dsl.inet.fi:/home/andrei/MySQL/FIXES/5.0/bug34427-rpl_vars_warn_slave_err · ace05448
      unknown authored
      into  mysql1000.dsl.inet.fi:/home/andrei/MySQL/MERGE/pushed.mysql-5.0-rpl-bug33329-extra_rollback
      
      ace05448
    • unknown's avatar
      Merge mysql1000.dsl.inet.fi:/home/andrei/MySQL/FIXES/5.0/bug32790-assert_mdel_norm_tables · 277e4b53
      unknown authored
      into  mysql1000.dsl.inet.fi:/home/andrei/MySQL/MERGE/pushed.mysql-5.0-rpl-bug33329-extra_rollback
      
      277e4b53
  6. 11 Feb, 2008 3 commits
    • unknown's avatar
      Problem: pushbuild has sporadic errors during startup of · bbef4be3
      unknown authored
      rpl_ndb_rep_ignore
      Reason: previous test, rpl_ndb_2multi_eng, does not sync slave with master
      after cleanup, so tables are sometimes left on slave
      Fix: sync_slave_with_master
      
      
      mysql-test/extra/rpl_tests/rpl_ndb_2multi_eng.test:
        Wait until drop table is replicated to slave before terminating. (The
        extra table caused sporadic errors in the next test case, causing
        pushbuild to fail.)
      bbef4be3
    • unknown's avatar
      Bug#27877 incorrect german order in utf8_general_ci · 3d8b763e
      unknown authored
      Problem: incorrect sort order for "U+00DF SHARP S".
      Fix: changing sort order for U+00DF to be equal to 's',
      like the manual says.
      
      
      
      mysql-test/r/ctype_latin1_de.result:
        Adding test
      mysql-test/r/ctype_uca.result:
        Adding test
      mysql-test/r/ctype_ucs.result:
        Adding test
      mysql-test/r/ctype_utf8.result:
        Adding test
      mysql-test/t/ctype_latin1_de.test:
        Adding test
      mysql-test/t/ctype_uca.test:
        Adding test
      mysql-test/t/ctype_ucs.test:
        Adding test
      mysql-test/t/ctype_utf8.test:
        Adding test
      strings/ctype-utf8.c:
        Changing weight for "U+00DF SHARP S"
      mysql-test/include/ctype_german.inc:
        New BitKeeper file ``mysql-test/include/ctype_german.inc''
      3d8b763e
    • unknown's avatar
      Merge mysql1000.dsl.inet.fi:/home/andrei/MySQL/FIXES/5.0/bug33329-extra_rollback_when_innodb_in · 2b85218b
      unknown authored
      into  mysql1000.dsl.inet.fi:/home/andrei/MySQL/MERGE/mysql-5.0-rpl-bug33329-extra_rollback
      
      2b85218b
  7. 09 Feb, 2008 1 commit
  8. 08 Feb, 2008 2 commits
    • unknown's avatar
      bug#34427 slave misses rendezvous in rpl_variables · 957bd4f7
      unknown authored
      There was no instruction in the test that enforces the slave successfully connect
      to the master.
      The way the test was been written allowed the slave to had been late for rendezvous 
      so that about-connecting time queries to the master failed and are error-logged
      to had been seen in Warnings of pb.
      
      Fixed with adding a sychronization primitive to the test.
      No test case is possible, observe error logs on pb.
      
      Todo: revise need of rpl_report.pl's rules due to failing execution of
      queries from get_master_verion_and_clock().
      Any test should try to use a synchornization primitive like the current fix
      makes and do not let the slave to miss successful connecting.
      
      
      mysql-test/t/rpl_variables.test:
        Completing the test's body with a synch macro to not let
        the slave to miss the successful connecting to the master.
      957bd4f7
    • unknown's avatar
      BUG#33247: mysqlbinlog does not clean up after itself on abnormal termination · 42d84bc1
      unknown authored
      Problem: mysqlbinlog does not free memory if an error happens.
      Fix: binlog-processing functions do not call exit() anymore. Instead, they
      print an error and return an error code. Error codes are propagated all
      the way back to main, and all allocated memory is freed on the way.
      
      
      client/mysqlbinlog.cc:
        - New error handling policy: functions processing binlogs don't just
          exit() anymore. Instead, they print a message and return an error
          status.
        - New policy for the global `mysql' and `glob_description_event': these
          are not passed as parameters anymore. The global pointer is used
          instead.
        - More error situations are detected and reported.
        - Better error messages: the program never terminates with exit status 1
          without explanation any more. Fixed spelling errors. Use consistent
          format of messages (a single line beginning with "ERROR: " or
          "WARNING: " and ending with "." is printed to stderr.)
        - New memory handling: memory is always freed on program termination.
        - Better comments: more functions are explained, doxygen is used, and
          more precise formulations in some existing comments.
      mysql-test/suite/binlog/r/binlog_base64_flag.result:
        Result file updated since output format of mysqlbinlog changed while the
        test was disabled.
      mysql-test/suite/binlog/t/binlog_killed.test:
        Mysqlbinlog now works as described when the binlog is open. Hence, the
        --force-if-open flag must be passed
      mysql-test/suite/binlog/t/binlog_killed_simulate.test:
        Mysqlbinlog now works as described when the binlog is open. Hence, the
        --force-if-open flag must be passed
      mysql-test/suite/binlog/t/disabled.def:
        Now that mysqlbinlog cleans up after itself on abnormal termination, we
        can enable this test again.
      42d84bc1
  9. 07 Feb, 2008 6 commits
    • unknown's avatar
      WL#4078: Document binary format of binlog entries · 9ffce940
      unknown authored
      Documented Table_map_log_event and packed integer format. Improved
      other documentation. No change outside comments.
      
      
      sql/log_event.h:
        Documented Table_map_log_event and packed integer format. Improved
        other documentation. No change outside comments.
      9ffce940
    • unknown's avatar
      Merge dl145h.mysql.com:/data0/mkindahl/mysql-5.1 · c57b2995
      unknown authored
      into  dl145h.mysql.com:/data0/mkindahl/mysql-5.1-rpl
      
      
      mysql-test/suite/binlog/t/binlog_old_versions.test:
        Auto merged
      c57b2995
    • unknown's avatar
      Renaming some saved binary log files to avoid 99 characters · a2e6846d
      unknown authored
      limit for v7 tar.
      
      
      mysql-test/suite/binlog/std_data/ver_5_1_17.001:
        Rename: mysql-test/suite/binlog/std_data/binlog_old_version_5_1_17.000001 -> mysql-test/suite/binlog/std_data/ver_5_1_17.001
      mysql-test/suite/binlog/std_data/ver_5_1_23.001:
        Rename: mysql-test/suite/binlog/std_data/binlog_old_version_5_1_23.000001 -> mysql-test/suite/binlog/std_data/ver_5_1_23.001
      mysql-test/suite/binlog/std_data/ver_5_1-telco.001:
        Rename: mysql-test/suite/binlog/std_data/binlog_old_version_5_1-telco.000001 -> mysql-test/suite/binlog/std_data/ver_5_1-telco.001
      mysql-test/suite/binlog/std_data/ver_5_1-wl2325_r.001:
        Rename: mysql-test/suite/binlog/std_data/binlog_old_version_5_1-wl2325_row.000001 -> mysql-test/suite/binlog/std_data/ver_5_1-wl2325_r.001
      mysql-test/suite/binlog/std_data/ver_5_1-wl2325_s.001:
        Rename: mysql-test/suite/binlog/std_data/binlog_old_version_5_1-wl2325_stm.000001 -> mysql-test/suite/binlog/std_data/ver_5_1-wl2325_s.001
      mysql-test/suite/binlog/std_data/bug32407.001:
        Rename: mysql-test/suite/binlog/std_data/binlog-bug32407.000001 -> mysql-test/suite/binlog/std_data/bug32407.001
      a2e6846d
    • unknown's avatar
      Merge dl145h.mysql.com:/data0/mkindahl/mysql-5.1 · 8b281444
      unknown authored
      into  dl145h.mysql.com:/data0/mkindahl/mysql-5.1-rpl
      
      
      sql/slave.cc:
        Auto merged
      8b281444
    • unknown's avatar
      Disabling declaration of debug variable for non-debug builds. · ed1bcc02
      unknown authored
      
      sql/slave.cc:
        Disabling declaration in non-debug builds.
      ed1bcc02
    • unknown's avatar
      Merge dl145h.mysql.com:/data0/mkindahl/mysql-5.1 · fb005c4d
      unknown authored
      into  dl145h.mysql.com:/data0/mkindahl/mysql-5.1-rpl
      
      
      sql/slave.cc:
        Auto merged
      sql/sql_binlog.cc:
        Auto merged
      fb005c4d
  10. 06 Feb, 2008 4 commits
    • unknown's avatar
      Merge riska.(none):/home/sven/bk/b34355-backslash_in_path_name_under_win/5.0-rpl · 7612771d
      unknown authored
      into  riska.(none):/home/sven/bk/b34355-backslash_in_path_name_under_win/5.1-new-rpl
      
      
      client/mysqlbinlog.cc:
        Auto merged
      sql/slave.cc:
        Auto merged
      sql/slave.h:
        Auto merged
      7612771d
    • unknown's avatar
      Replace windows path separator backslash by unix path separator forward · 270e2f3c
      unknown authored
      slash in filenames also for Create_file_log_event.
      
      
      client/mysqlbinlog.cc:
        BUG#34355: mysqlbinlog outputs backslash as path separator for 4.1 binlogs
        Problem: When the windows version of mysqlbinlog reads 4.1 binlogs
        containing LOAD DATA INFILE, it outputs backslashes as path separators in
        filenames. However, the output is typically piped to a client, and client
        expects forward slashes.
        Fix: Replace '\\' by '/' in filenames.
      270e2f3c
    • unknown's avatar
      Patch to eliminate some valgrind warnings in debug printout code. · 14fb6f70
      unknown authored
      
      sql/rpl_rli.cc:
        Adding variable to mark an instance of Relay_log_info as fake.
      sql/rpl_rli.h:
        Adding variable to mark an instance of Relay_log_info as fake.
      sql/slave.cc:
        Not printing debug information if we are working with a fake
        instance of Relay_log_info. This because the result of calling
        update is nonsense, and trying to print it generates valgrind
        warnings.
      sql/sql_binlog.cc:
        Marking newly created instance of Relay_log_info as a fake instance.
      14fb6f70
    • unknown's avatar
      Removing duplicate code from mysql-test-run.pl · 1381594b
      unknown authored
      
      mysql-test/mysql-test-run.pl:
        Removing duplicate code.
      1381594b
  11. 05 Feb, 2008 6 commits
    • unknown's avatar
      Updating result file for rpl_loaddata_map. · 19025ba0
      unknown authored
      
      mysql-test/suite/rpl/r/rpl_loaddata_map.result:
        Result change.
      19025ba0
    • unknown's avatar
      Fixes to make tests pass on vanilla build. · 0ea839e5
      unknown authored
      
      mysql-test/suite/binlog/t/binlog_old_versions.test:
        Adding --local-load to 'mysqlbinlog' and --local-infile=1 to 'mysql'.
      0ea839e5
    • unknown's avatar
      Bug #34305 show slave status handling segfaults when slave io is about · 89e828b4
      unknown authored
          to leave
      
      The artifact was caused by
      a flaw in concurrent accessing the slave's io thd by
      the io itself and a handling show slave status thread.
      Namely, show_master_info did not acquire mi->run_lock mutex that is
      specified for mi->io_thd member.
      
      Fixed with deploying the mutex locking and unlocking. The mutex is kept
      short time and without interleaving with mi->data_lock mutex.
      
      Todo: to report and fix an issue with 
          sys_var_slave_skip_counter::{methods} 
      seem to acquire incorrectly
           active_mi->rli.run_lock
      instead of the specified
           active_mi->rli.data_lock
      
      A test case is difficult to compose, so rpl_packet should continue serving
      as the indicator.
      
      
      sql/slave.cc:
        implementing a TODO left at 4.1 time:
        mending access to mi->io_thd with the specified mutex;
      sql/slave.h:
        adding a member name to the list of that run_lock guards.
      89e828b4
    • unknown's avatar
      Merge mkindahl@bk-internal.mysql.com:/home/bk/mysql-5.1-new-rpl · 543095c2
      unknown authored
      into  dl145h.mysql.com:/data0/mkindahl/mysql-5.1-rpl
      
      
      include/my_base.h:
        Auto merged
      mysql-test/include/show_binlog_events2.inc:
        Auto merged
      mysql-test/suite/binlog/t/binlog_old_versions.test:
        Auto merged
      mysql-test/suite/rpl/r/rpl_row_tabledefs_2myisam.result:
        Auto merged
      mysql-test/suite/rpl/r/rpl_row_tabledefs_3innodb.result:
        Auto merged
      mysql-test/suite/rpl/t/rpl_loaddata_map-master.opt:
        Auto merged
      mysql-test/suite/rpl/t/rpl_loaddata_map-slave.opt:
        Auto merged
      sql/log_event_old.cc:
        Auto merged
      sql/sql_binlog.cc:
        Auto merged
      mysql-test/suite/rpl/r/rpl_loaddata_map.result:
        Manual merge.
      mysql-test/suite/rpl/r/rpl_user.result:
        Manual merge.
      mysql-test/suite/rpl/t/rpl_loaddata_map.test:
        Manual merge.
      mysql-test/suite/rpl/t/rpl_user.test:
        Manual merge.
      sql/log_event.cc:
        Manual merge.
      sql/rpl_record.cc:
        Manual merge.
      543095c2
    • unknown's avatar
      Adding some more filtering for tests that kills queries and · aad6ca0f
      unknown authored
      cause sporadic, but benign, errors.
      
      
      mysql-test/lib/mtr_report.pl:
        Filtering out sporadic but benign error in rpl_err_ignoredtable.
      aad6ca0f
    • unknown's avatar
      Disabling test case binlog_old_version for embedded server. · ca9da416
      unknown authored
      
      mysql-test/lib/mtr_report.pl:
        Ignoring benign error message (killing query causes unpredictable
        error messages).
      mysql-test/suite/binlog/t/binlog_old_versions.test:
        Disabling test case for embedded server.
      sql/rpl_rli.cc:
        Inititializing uninitialized variable causing valgrind warning.
      ca9da416
  12. 04 Feb, 2008 2 commits
    • unknown's avatar
      Bug#33329 extraneous ROLLBACK in binlog on connection · a3d820e7
      unknown authored
                  does not use trans tables
      
      There had been two issues.
      Rollback statement was recorded in binlog even though a multi-update
      had not modified any non-transactional table.
      The reason for this artifact was a false initial value of multi_update::transactional_tables.
      Yet another artifact that explained on the bug page is that 
      `ha_autocommit_or_rollback' works differently depending on whether
      a transaction engine has been compiled in. 
      
      Fixed: with setting multi_update::transactional_tables to zero at initialization
      time. Multi-update on non-trans table won't cause ROLLBACK in binlog with
      either compilation option.
      
      The 2nd mentioned artifact comprises a self-standing issue (to be reported
      separately).
      
      
      mysql-test/r/multi_update.result:
        results changed - there is no ROLLBACK in binlog anymore as it should be
      sql/sql_update.cc:
        A wrong assumption on that there were modified transactional table,
        which is nonsense at the very beginning of the query execution.
      a3d820e7
    • unknown's avatar
      Bug #32790 crash in trigger.test with InnoDB for a table · ec529c48
      unknown authored
      the reason for the failure were incorrect asserts.
      
      Removing asserts altogether as there is no the implication does not hold
      (as explained in the comments for the file).
      
      
      sql/sql_delete.cc:
        removing two asserts because they can not hold basing on the definition
        of `normal_tables'. The one does not specify in a non-transactional table,
        which must be in the list of tables to be deleted, is modified indeed.
        So, it's possible to have normal_tables == true and deleted == true both
        but that would be yet a transactional table got modified (and then
        thd->transaction.stmt.modified_non_trans_table remains false default).
      ec529c48