List:Commits« Previous MessageNext Message »
From:Gleb Shchepa Date:December 3 2009 8:04pm
Subject:bzr push into mysql-5.1-bugteam branch (gshchepa:3242 to 3243)
View as plain text  
 3243 Gleb Shchepa	2009-12-03
      Bug #38883: thd_security_context is not thread safe, crashes?
      After-push minor code cleanup for WL 2360: unnecessary external
      reference to LOCK_thread_count has been removed from

 3242 lars-erik.bjork@stripped	2009-12-03
      This is a patch for bug#41569.
      "mysql_upgrade (ver 5.1) add 3 fields to mysql.proc table but does
      not set values".
      mysql_upgrade (ver 5.1) adds 3 fields (character_set_client, 
      collation_connection and db_collation) to the mysql.proc table, but 
      does not set any values. When we run stored procedures, which were 
      created with mysql 5.0, a warning is logged into the error log.
      The solution to this is for mysql_upgrade to set default best guess
      values for these fields. A warning is also written during upgrade, to
      make the user aware that default values are set.
     @ client/mysql_upgrade.c
        Result lines which start with "WARNING" are passed through to the output. 
        This way we have a way of triggering WARNING-messages during upgrade 
        directly from the .sql-script.
     @ mysql-test/r/mysql_upgrade.result
        Expected result of the test.
     @ mysql-test/t/mysql_upgrade.test
        Added a test-case for the bug.
     @ scripts/mysql_system_tables_fix.sql
        The new fields are populated, and warnings are written.

=== modified file 'storage/innobase/handler/'
--- a/storage/innobase/handler/	2009-12-01 10:38:40 +0000
+++ b/storage/innobase/handler/	2009-12-03 19:38:09 +0000
@@ -40,12 +40,6 @@ have disabled the InnoDB inlining in thi
 #include "ha_innodb.h"
 #include <mysql/plugin.h>
-/* This is needed because of Bug #3596. Let us hope that pthread_mutex_t
-is defined the same in both builds: the MySQL server and the InnoDB plugin. */
-extern pthread_mutex_t LOCK_thread_count;
-#endif /* MYSQL_SERVER */
 /** to protect innobase_open_files */
 static pthread_mutex_t innobase_share_mutex;
 /** to force correct commit order in binlog */

=== modified file 'storage/innodb_plugin/handler/'
--- a/storage/innodb_plugin/handler/	2009-12-01 10:38:40 +0000
+++ b/storage/innodb_plugin/handler/	2009-12-03 19:38:09 +0000
@@ -110,9 +110,6 @@ extern "C" {
 #  define MYSQL_PLUGIN_IMPORT /* nothing */
 # endif /* MYSQL_PLUGIN_IMPORT */
-/* This is needed because of Bug #3596. Let us hope that pthread_mutex_t
-is defined the same in both builds: the MySQL server and the InnoDB plugin. */
-extern MYSQL_PLUGIN_IMPORT pthread_mutex_t LOCK_thread_count;
 #if MYSQL_VERSION_ID < 50124
 /* this is defined in mysql_priv.h inside #ifdef MYSQL_SERVER

Attachment: [text/bzr-bundle] bzr/
bzr push into mysql-5.1-bugteam branch (gshchepa:3242 to 3243)Bug#38883Gleb Shchepa3 Dec