1. 19 Feb, 2008 2 commits
  2. 18 Feb, 2008 3 commits
    • unknown's avatar
      Merge from 5.1-build of the "my_bool for C" changeset. · 340f3369
      unknown authored
      Removing "#define bool BOOL" as it's dangerous.
      
      
      include/config-win.h:
        This #define was confusing: a) it changes the semantics of bool but
        only on Windows (C++ standard says that casting int to bool to int
        yields 0 or 1, but if using BOOL (which is int) this breaks, see
        old BUG 11567 "Variable 'foreign_key_checks', and other variables,
        differs binlog tests") b) it is not needed in C++ which has a bool type
        c) it is not needed in C where we must use my_bool as bool either does
        not exist or changes size between C and C++.
      storage/maria/ma_check.c:
        my_bool for C
      storage/maria/ma_dynrec.c:
        my_bool for C
      storage/maria/ma_search.c:
        my_bool for C
      storage/maria/ma_write.c:
        my_bool for C
      storage/maria/maria_def.h:
        my_bool for C
      storage/maria/maria_pack.c:
        my_bool for C
      340f3369
    • unknown's avatar
      Merge gbichot4.local:/home/mysql_src/mysql-5.1-build-gca · 9661026a
      unknown authored
      into  gbichot4.local:/home/mysql_src/mysql-maria-monty
      
      
      client/mysqldump.c:
        Auto merged
      client/mysqltest.c:
        Auto merged
      extra/replace.c:
        Auto merged
      include/my_sys.h:
        Auto merged
      libmysql/libmysql.c:
        Auto merged
      mysys/my_getopt.c:
        Auto merged
      mysys/thr_lock.c:
        Auto merged
      sql/set_var.cc:
        Auto merged
      sql/sql_plugin.cc:
        Auto merged
      storage/myisam/mi_check.c:
        Auto merged
      storage/myisam/mi_dynrec.c:
        Auto merged
      storage/myisam/mi_search.c:
        Auto merged
      storage/myisam/mi_update.c:
        Auto merged
      storage/myisam/mi_write.c:
        Auto merged
      storage/myisam/myisamlog.c:
        Auto merged
      storage/myisam/myisampack.c:
        Auto merged
      tests/mysql_client_test.c:
        Auto merged
      include/my_global.h:
        merge
      storage/myisam/myisamdef.h:
        merge
      9661026a
    • unknown's avatar
      Fix for server bug experienced in Maria (wrong "Truncated incorrect <var_name> · e2103aaf
      unknown authored
      value" error even though the value was correct): a C function in my_getopt.c
      was taking bool* in parameter and was called from C++ sql_plugin.cc,
      but on some Mac OS X sizeof(bool) is 1 in C and 4 in C++, giving funny
      mismatches. Fixed, all other occurences of bool in C are removed, future
      ones are blocked by a "C-bool-catcher" in my_global.h (use my_bool).
      
      
      client/mysqldump.c:
        my_bool for C
      client/mysqltest.c:
        my_bool for C
      extra/replace.c:
        my_bool for C
      include/my_getopt.h:
        my_bool for C
      include/my_global.h:
        Prevent people from using bool in C, it causes real bugs.
      include/my_sys.h:
        my_bool for C
      include/my_time.h:
        my_bool for C
      include/thr_lock.h:
        my_bool for C
      libmysql/libmysql.c:
        my_bool for C
      mysys/charset.c:
        my_bool for C
      mysys/my_getopt.c:
        my_bool for C
      mysys/queues.c:
        my_bool for C
      mysys/thr_lock.c:
        my_bool for C
      regex/reginit.c:
        my_bool for C
      sql/set_var.cc:
        C functions use my_bool so we must use my_bool too.
      sql/sql_plugin.cc:
        C functions use my_bool so we must use my_bool too.
        This fixes a real observed bug of Maria, because on some Mac OS X,
        sizeof(bool) is 1 in C and 4 in C++, so the bool* does wrong.
        Removing useless line.
      storage/heap/hp_update.c:
        my_bool for C
      storage/myisam/mi_check.c:
        my_bool for C
      storage/myisam/mi_dynrec.c:
        my_bool for C
      storage/myisam/mi_search.c:
        my_bool for C
      storage/myisam/mi_update.c:
        my_bool for C
      storage/myisam/mi_write.c:
        my_bool for C
      storage/myisam/myisamdef.h:
        my_bool for C
      storage/myisam/myisamlog.c:
        my_bool for C
      storage/myisam/myisampack.c:
        my_bool for C
      tests/mysql_client_test.c:
        my_bool for C
      unittest/mysys/bitmap-t.c:
        my_bool for C
      vio/viosslfactories.c:
        my_bool for C
      e2103aaf
  3. 15 Feb, 2008 1 commit
    • unknown's avatar
      Fix for maria.test failure in embedded server (test split) · 05f3e1c5
      unknown authored
      
      mysql-test/r/maria.result:
        result update
      mysql-test/t/maria.test:
        The embedded server translates INSERT DELAYED to plain INSERT
        so does not produce an error; moving this test to a new test
        which is not run for the embedded server.
      mysql-test/r/maria_notembedded.result:
        result is ok
      mysql-test/t/maria_notembedded.test:
        New test for cases where embedded server is different.
      05f3e1c5
  4. 13 Feb, 2008 8 commits
    • unknown's avatar
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-maria · 95a04ca9
      unknown authored
      into  mysql.com:/home/my/mysql-maria
      
      95a04ca9
    • unknown's avatar
      skip dbug tests if DBUG_OFF · 707835f0
      unknown authored
      
      include/my_dbug.h:
        warnings
      707835f0
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-maria · f03d3a55
      unknown authored
      into  mysql.com:/home/my/mysql-maria
      
      f03d3a55
    • unknown's avatar
      Fixed compiler warnings in a lot of files · dff4e5e1
      unknown authored
      Added IMPOSSIBLE_RESULT to avoid compiler warnings when using (Item_result) -1 as a dummy value
      Changed PAGE_SIZE to TEST_PAGE_SIZE to avoid compiler errors on systems where PAGE_SIZE is defined
      
      
      client/get_password.c:
        Fixed compiler warning
      cmd-line-utils/readline/bind.c:
        Fixed compiler warning
      cmd-line-utils/readline/chardefs.h:
        Fixed compiler warning by adding marco to be used when largest_char is 255
      cmd-line-utils/readline/display.c:
        Fixed compiler warnings by removing not accessed variables
      cmd-line-utils/readline/histexpand.c:
        Fixed compiler warnings by removing not accessed variables
      cmd-line-utils/readline/history.c:
        Fixed compiler warnings by adding cast
      cmd-line-utils/readline/text.c:
        Fixed compiler warnings by removing not accessed variables and adding casts
      dbug/dbug.c:
        Fixed compiler warnings by changing types
      include/mysql_com.h:
        Added IMPOSSIBLE_RESULT to avoid compiler warnings when using (Item_result) -1 as a dummy value
      libmysql/libmysql.c:
        Fixed compiler warning
      mysql-test/t/query_cache_debug.test:
        Mark test as BIG as it uses a lot of memory
      mysys/mf_iocache2.c:
        Fixed compiler warnings by adding cast
      sql/event_data_objects.cc:
        Fixed compiler warnings by removing not used code
      sql/events.cc:
        Fixed compiler warnings by removing not used code
      sql/field.cc:
        Fixed compiler warnings by adding cast and removed not accessed variables
      sql/ha_partition.cc:
        Fixed compiler warnings by removing not used code
      sql/item.cc:
        Fixed compiler warnings by removing not accessed variables
        Use IMPOSSIBLE_RESULT instead of (Item_result)-1
      sql/item_cmpfunc.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/item_func.cc:
        Fixed compiler warnings by removing not used code and not accessed variables
        Added IMPOSSIBLE_RESULT
      sql/item_subselect.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/item_xmlfunc.cc:
        Fixed forgotten setting of xpath->error
      sql/log.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/log_event.cc:
        Added IMPOSSIBLE_RESULT into switch
        Fixed wrong usage of DBUG_ASSERT(1)
        Removed always true DBUG_ASSERT()
      sql/mysqld.cc:
        Fixed compiler warnings by adding casts for ULONG_MAX
      sql/opt_sum.cc:
        Fixed compiler warnings by removing not used code
        Removed wrong DBUG_ASSERT()
      sql/partition_info.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/rpl_injector.h:
        Removed always true part from DBUG_ASSERT() to remove compiler warning
      sql/spatial.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/sql_acl.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/sql_base.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/sql_cache.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/sql_class.cc:
        Fixed compiler warnings by:
        - Removing always true part from DBUG_ASSERT()
        - Removing not used code
        - Added IMPOSSIBLE_RESULT into switch
      sql/sql_load.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/sql_parse.cc:
        Fixed compiler warnings by:
        - Removing not accessed variables
        - Removing always true part from DBUG_ASSERT()
        - Removing not used code
      sql/sql_plugin.cc:
        Added comment
      sql/sql_prepare.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/sql_show.cc:
        Fixed compiler warnings by using correct cast
      sql/sql_table.cc:
        Fixed compiler warnings by removing not used code and removing not accessed variables
      sql/table.cc:
        Fixed compiler warnings by removing not accessed variables
      sql/time.cc:
        Fixed wrong DBUG_ASSERT(1)
      storage/maria/unittest/Makefile.am:
        Changed PAGE_SIZE to TEST_PAGE_SIZE to avoid compiler errors on systems where PAGE_SIZE is defined
      storage/maria/unittest/ma_pagecache_consist.c:
        Changed PAGE_SIZE to TEST_PAGE_SIZE to avoid compiler errors on systems where PAGE_SIZE is defined
      storage/maria/unittest/ma_pagecache_single.c:
        Changed PAGE_SIZE to TEST_PAGE_SIZE to avoid compiler errors on systems where PAGE_SIZE is defined
      tests/mysql_client_test.c:
        Fixed compiler warnings by removing not accessed variables and changing types
      dff4e5e1
    • unknown's avatar
      compiler warning fixed · 8654bc4e
      unknown authored
      8654bc4e
    • unknown's avatar
      build/test failures on different platforms · 9e0565f4
      unknown authored
      
      include/atomic/rwlock.h:
        define MY_ATOMIC_MODE_RWLOCKS if this is the way we have to go
      mysys/lf_alloc-pin.c:
        no semicolon
      mysys/lf_hash.c:
        no semicolon
      storage/maria/lockman.c:
        no semicolon
      storage/maria/ma_loghandler.c:
        no semicolon
      unittest/mysys/my_atomic-t.c:
        powerpc is no better
        (condition could be a bit too broad, but hey, it's just a unit test)
      9e0565f4
    • unknown's avatar
      fix for embedded build · 991030af
      unknown authored
      991030af
  5. 11 Feb, 2008 14 commits
    • unknown's avatar
      Merge abelkin@bk-internal.mysql.com:/home/bk/mysql-maria · 022e7489
      unknown authored
      into  oleksandr-byelkins-powerbook-g4-15.local:/Users/bell/mysql/bk/mysql-maria
      
      022e7489
    • unknown's avatar
      Comparison between signed and unsigned fixed. · a3234486
      unknown authored
      
      storage/maria/ma_loghandler.c:
        Fixed comparison of signed and unsigned.
      a3234486
    • unknown's avatar
      Merge gbichot@bk-internal.mysql.com:/home/bk/mysql-maria · fdd7292c
      unknown authored
      into  gbichot4.local:/home/mysql_src/mysql-maria-monty
      
      fdd7292c
    • unknown's avatar
      fix for non-debug build · b7e94e75
      unknown authored
      b7e94e75
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-maria · 346e8da0
      unknown authored
      into  mysql.com:/home/my/mysql-maria
      
      346e8da0
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-maria · 02d64ee2
      unknown authored
      into  mysql.com:/home/my/mysql-maria
      
      02d64ee2
    • unknown's avatar
      Merge gbichot@bk-internal.mysql.com:/home/bk/mysql-maria · 78dcf2a3
      unknown authored
      into  gbichot4.local:/home/mysql_src/mysql-maria-monty
      
      78dcf2a3
    • unknown's avatar
      Add warning of TRANSACTIONAL=1 is used with handler that doesn't support it · 1c3a96ed
      unknown authored
      Added hton_name() and handler->engine_name() and use these when appropriate
      
      
      KNOWN_BUGS.txt:
        Added some of the known bugs
      mysql-test/r/maria.result:
        Added test for TRANSACTIONAL=1
      mysql-test/r/myisam.result:
        Added test for TRANSACTIONAL=1
      mysql-test/t/maria.test:
        Added test for TRANSACTIONAL=1
      mysql-test/t/myisam.test:
        Added test for TRANSACTIONAL=1
      sql/handler.cc:
        ha_resolve_storage_engine_name() -> hton_name()
      sql/handler.h:
        Added hton_name() and handler->engine_name()
      sql/set_var.cc:
        Use hton_name()
      sql/sql_table.cc:
        Add warning of TRANSACTIONAL=1 is used with handler that doesn't support it
        Indentation fixes
        ha_resolve_storage_engine_name() -> hton_name()
      sql/sql_tablespace.cc:
        ha_resolve_storage_engine_name() -> hton_name()
        Indentation fixes
      sql/sql_yacc.yy:
        ha_resolve_storage_engine_name() -> hton_name()
      sql/unireg.cc:
        ha_resolve_storage_engine_name() -> hton_name()
        Indentation fixes
      1c3a96ed
    • unknown's avatar
      Merge gbichot@bk-internal.mysql.com:/home/bk/mysql-maria · 4718f433
      unknown authored
      into  mysqlwin32.:C:/mysql-maria
      
      4718f433
    • unknown's avatar
      A new option for maria_chk: --zerofill-keep-lsn. This will be used · e5428802
      unknown authored
      by ma_test_recovery.pl when it happens that Recovery does not recreate
      pages exactly as they were at first run: this option will help us
      verify that the differences are in unimportant page pieces (those pieces
      will be zeroed by --zerofill-keep-lsn, but not the important LSNs).
      
      
      include/myisamchk.h:
        new zerofill flag for maria_chk
      storage/maria/ma_check.c:
        If T_ZEROFILL_KEEP_LSN, we don't zero out LSNs of data/index pages.
        Then the table is not movable. We still mark it zerofilled, it helps
        to know what was last done to the table.
      storage/maria/maria_chk.c:
        New option --zerofill-keep-lsn
      e5428802
    • unknown's avatar
      Merge gbichot@bk-internal.mysql.com:/home/bk/mysql-maria · 81d5bf4a
      unknown authored
      into  mysqlwin32.:C:/mysql-maria
      
      81d5bf4a
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-new-rpl · 64169931
      unknown authored
      into  janus.mylan:/usr/home/serg/Abk/mysql-maria
      
      
      mysql-test/mysql-test-run.pl:
        Auto merged
      sql/log_event.cc:
        Auto merged
      sql/slave.cc:
        Auto merged
      sql/slave.h:
        Auto merged
      sql/share/errmsg.txt:
        Auto merged
      include/my_base.h:
        merged
      64169931
    • 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
      Fix for failures of ma_control_file-t, ma_test_loghandler_purge-t, · 664ddb61
      unknown authored
      ma_test_loghandler_pagecache-t on Windows.
      
      
      storage/maria/ma_control_file.c:
        stat() is unreliable on Windows (does not reflect process' own writes)
      storage/maria/ma_loghandler.c:
        translog_set_lsn_for_files() didn't close its file descriptor; it was
        a real problem as non-closed files could not be purged.
        Same for translog_truncate_log() in case of error.
      storage/maria/unittest/ma_test_loghandler_pagecache-t.c:
        stat() is unreliable on Windows (does not reflect process' own writes)
      664ddb61
  6. 09 Feb, 2008 2 commits
  7. 08 Feb, 2008 8 commits
    • unknown's avatar
      Merge mysql.com:/home/kent/bk/maria/mysql-5.1-release · d406184e
      unknown authored
      into  mysql.com:/home/kent/bk/maria/mysql-maria
      
      
      extra/yassl/src/handshake.cpp:
        Auto merged
      scripts/make_binary_distribution.sh:
        Auto merged
      sql/handler.cc:
        Auto merged
      sql/log_event.cc:
        Auto merged
      d406184e
    • 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
    • unknown's avatar
      Correcting assertion · 6761c498
      unknown authored
      
      storage/maria/ha_maria.cc:
        start_stmt() is passed thr_lock_type (TL_*), not int F_*.
        Though tt does not seem to matter here, F_UNLCK == TL_UNLOCK == 0
        at least on Linux and Windows.
      6761c498
    • unknown's avatar
      ma_pagecache.c: · a6c7eb1b
      unknown authored
        SCCS merged
      
      
      storage/maria/ma_pagecache.c:
        SCCS merged
      a6c7eb1b
    • unknown's avatar
      BUG#34089 "Maria crash on LOAD INDEX after FLUSH TABLES" · 593528fb
      unknown authored
      a typo had not disable LOAD INDEX for Maria.
      BUG#34161 "freeing maria tmp table after fetching rows using prepared statements - crashes"
      running the testcase I was not able to repeat the same crash but
      found a misplaced assertion giving false positive, fixed here.
      
      
      storage/maria/ha_maria.cc:
        important typo
      storage/maria/ma_pagecache.c:
        The assertion on block's type should be after read_block(). Otherwise
        it can give false alarms.
      593528fb
    • unknown's avatar
      Merge gbichot@bk-internal.mysql.com:/home/bk/mysql-maria · dc4b257a
      unknown authored
      into  mysqlwin32.:C:/mysql-maria
      
      dc4b257a
    • unknown's avatar
      Fix for bug in log handler observed when running maria-big · 539bbaae
      unknown authored
      and then maria-connect under Windows 32-bit Visual Studio 2003.
      Build of all Maria unit tests under Windows.
      
      
      storage/maria/ma_loghandler.c:
        Assertion contributed by Sanja. Working around my_stat() bug on Windows
        (st_size is long i.e. 32-bit signed, the file was slightly bigger than
        2GB, so st_size was negative, TRANSLOG_PAGE_SIZE was int, so the
        comparison was done using long, so st_size was still negative in
        the comparison, which was incorrect). Two fixes: cast st_size to uint32
        (belt) and make TRANSLOG_PAGE_SIZE unsigned (suspenders).
        Of course 'long' is not enough for files >=4GB, but all log files of
        Maria are <4GB. The general problem of my_stat() has been passed on to
        our Windows team.
      storage/maria/ma_loghandler.h:
        use unsigned ints for constants which represent sizes of pages, files...
      storage/maria/unittest/CMakeLists.txt:
        a few unit tests were not built under Windows
      539bbaae
    • unknown's avatar
      Postmerge mac compiler warnings fixed. · 9637affb
      unknown authored
      9637affb
  8. 07 Feb, 2008 2 commits
    • unknown's avatar
      Merge oleksandr-byelkins-powerbook-g4-15.local:/Users/bell/mysql/bk/mysql-maria · 3bfa89bd
      unknown authored
      into  oleksandr-byelkins-powerbook-g4-15.local:/Users/bell/mysql/bk/work-maria-wrn
      
      
      storage/maria/ma_blockrec.c:
        Auto merged
      storage/maria/ma_check.c:
        Auto merged
      storage/maria/ma_control_file.c:
        Auto merged
      storage/maria/ma_key_recover.c:
        Auto merged
      storage/maria/ma_page.c:
        Auto merged
      storage/maria/ma_pagecache.c:
        Auto merged
      storage/maria/ma_recovery.c:
        Auto merged
      3bfa89bd
    • unknown's avatar
      Mac compiler warnings fixed. · 1dec5f9e
      unknown authored
      
      include/maria.h:
        mac compiler warnings fixed.
      storage/maria/ma_bitmap.c:
        mac compiler warnings fixed.
      storage/maria/ma_blockrec.c:
        mac compiler warnings fixed.
      storage/maria/ma_check.c:
        mac compiler warnings fixed.
      storage/maria/ma_control_file.c:
        mac compiler warnings fixed.
      storage/maria/ma_create.c:
        mac compiler warnings fixed.
      storage/maria/ma_delete.c:
        mac compiler warnings fixed.
      storage/maria/ma_ft_boolean_search.c:
        mac compiler warnings fixed.
      storage/maria/ma_page.c:
        mac compiler warnings fixed.
      storage/maria/ma_pagecache.c:
        mac compiler warnings fixed.
      storage/maria/ma_recovery.c:
        mac compiler warning fixed.
      storage/maria/ma_rt_test.c:
        mac compiler warnings fixed.
      storage/maria/ma_search.c:
        mac compiler warning fixed.
      storage/maria/ma_write.c:
        mac compiler warnings fixed.
      storage/maria/unittest/ma_control_file-t.c:
        mac compiler warnings fixed.
      storage/maria/unittest/ma_pagecache_consist.c:
        mac compiler warnings fixed.
      storage/maria/unittest/ma_pagecache_single.c:
        mac compiler warnings fixed.
      storage/maria/unittest/ma_test_loghandler_pagecache-t.c:
        mac compiler warning fixed.
      storage/maria/unittest/test_file.c:
        mac compiler warning fixed.
      1dec5f9e