summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorunknown <bar@mysql.com>2007-04-09 18:06:27 +0500
committerunknown <bar@mysql.com>2007-04-09 18:06:27 +0500
commit6d9d64baab8537e465659daab77d2a9fd1a286ff (patch)
treece446d2a642cfd3838c2a928a265568cd482c9be
parentec9b5eca9cee569eb4c1442cb0ab47c72dac818d (diff)
parent2e302abc80ea82ba145289fb6951ccbbf51076b6 (diff)
downloadmariadb-git-6d9d64baab8537e465659daab77d2a9fd1a286ff.tar.gz
Merge mysql.com:/home/bar/mysql-5.0-rpl
into mysql.com:/home/bar/mysql-5.1-new-rpl client/mysql.cc: Auto merged mysql-test/r/variables.result: Auto merged mysql-test/t/variables.test: Auto merged sql/mysql_priv.h: Auto merged sql/set_var.cc: Auto merged sql/sql_class.cc: Auto merged sql/mysqld.cc: SCCS merged
-rw-r--r--client/mysql.cc11
-rw-r--r--mysql-test/r/client_xml.result14
-rw-r--r--mysql-test/r/variables.result31
-rw-r--r--mysql-test/t/client_xml.test2
-rw-r--r--mysql-test/t/variables.test24
-rw-r--r--sql/mysql_priv.h1
-rw-r--r--sql/mysqld.cc19
-rw-r--r--sql/set_var.cc14
-rw-r--r--sql/sql_class.cc1
9 files changed, 112 insertions, 5 deletions
diff --git a/client/mysql.cc b/client/mysql.cc
index d2c7bd8812c..e54732d8ac6 100644
--- a/client/mysql.cc
+++ b/client/mysql.cc
@@ -2104,6 +2104,17 @@ com_go(String *buffer,char *line __attribute__((unused)))
if (!mysql_num_rows(result) && ! quick && !column_types_flag)
{
strmov(buff, "Empty set");
+ if (opt_xml)
+ {
+ /*
+ We must print XML header and footer
+ to produce a well-formed XML even if
+ the result set is empty (Bug#27608).
+ */
+ init_pager();
+ print_table_data_xml(result);
+ end_pager();
+ }
}
else
{
diff --git a/mysql-test/r/client_xml.result b/mysql-test/r/client_xml.result
index 6a148954fcd..ed5e8f2c1b8 100644
--- a/mysql-test/r/client_xml.result
+++ b/mysql-test/r/client_xml.result
@@ -71,4 +71,18 @@ insert into t1 values (1, 2, 'a&b a<b a>b');
<field name="NULL" xsi:nil="true" />
</row>
</resultset>
+<?xml version="1.0"?>
+
+<resultset statement="select 1 limit 0
+" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"></resultset>
+--------------
+select 1 limit 0
+--------------
+
+<?xml version="1.0"?>
+
+<resultset statement="select 1 limit 0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"></resultset>
+Empty set
+
+Bye
drop table t1;
diff --git a/mysql-test/r/variables.result b/mysql-test/r/variables.result
index 6927f07da6b..8d601596648 100644
--- a/mysql-test/r/variables.result
+++ b/mysql-test/r/variables.result
@@ -753,6 +753,37 @@ set lc_time_names=0;
select @@lc_time_names;
@@lc_time_names
en_US
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+en_US en_US
+set @@global.lc_time_names=fr_FR;
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+fr_FR en_US
+New connection
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+fr_FR fr_FR
+set @@lc_time_names=ru_RU;
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+fr_FR ru_RU
+Returnung to default connection
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+fr_FR en_US
+set lc_time_names=default;
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+fr_FR fr_FR
+set @@global.lc_time_names=default;
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+en_US fr_FR
+set @@lc_time_names=default;
+select @@global.lc_time_names, @@lc_time_names;
+@@global.lc_time_names @@lc_time_names
+en_US en_US
set @test = @@query_prealloc_size;
set @@query_prealloc_size = @test;
select @@query_prealloc_size = @test;
diff --git a/mysql-test/t/client_xml.test b/mysql-test/t/client_xml.test
index 017b7a1569a..8ee63a6131b 100644
--- a/mysql-test/t/client_xml.test
+++ b/mysql-test/t/client_xml.test
@@ -17,5 +17,7 @@ insert into t1 values (1, 2, 'a&b a<b a>b');
--exec $MYSQL --xml test -e "select 1 > 2 from dual"
--exec $MYSQL --xml test -e "select 1 & 3 from dual"
--exec $MYSQL --xml test -e "select null from dual"
+--exec $MYSQL --xml test -e "select 1 limit 0"
+--exec $MYSQL --xml test -vv -e "select 1 limit 0"
drop table t1;
diff --git a/mysql-test/t/variables.test b/mysql-test/t/variables.test
index d381ed62bc7..12d8a89fd08 100644
--- a/mysql-test/t/variables.test
+++ b/mysql-test/t/variables.test
@@ -540,6 +540,30 @@ set lc_time_names=0;
select @@lc_time_names;
#
+# Bug #22648 LC_TIME_NAMES: Setting GLOBAL has no effect
+#
+select @@global.lc_time_names, @@lc_time_names;
+set @@global.lc_time_names=fr_FR;
+select @@global.lc_time_names, @@lc_time_names;
+--echo New connection
+connect (con1,localhost,root,,);
+connection con1;
+select @@global.lc_time_names, @@lc_time_names;
+set @@lc_time_names=ru_RU;
+select @@global.lc_time_names, @@lc_time_names;
+disconnect con1;
+connection default;
+--echo Returnung to default connection
+select @@global.lc_time_names, @@lc_time_names;
+set lc_time_names=default;
+select @@global.lc_time_names, @@lc_time_names;
+set @@global.lc_time_names=default;
+select @@global.lc_time_names, @@lc_time_names;
+set @@lc_time_names=default;
+select @@global.lc_time_names, @@lc_time_names;
+
+
+#
# Bug #13334: query_prealloc_size default less than minimum
#
set @test = @@query_prealloc_size;
diff --git a/sql/mysql_priv.h b/sql/mysql_priv.h
index 2bc8c2071d2..bb5c2f51a76 100644
--- a/sql/mysql_priv.h
+++ b/sql/mysql_priv.h
@@ -147,6 +147,7 @@ typedef struct my_locale_st
extern MY_LOCALE my_locale_en_US;
extern MY_LOCALE *my_locales[];
+extern MY_LOCALE *my_default_lc_time_names;
MY_LOCALE *my_locale_by_name(const char *name);
MY_LOCALE *my_locale_by_number(uint number);
diff --git a/sql/mysqld.cc b/sql/mysqld.cc
index 5031496158b..7c7d9dc923f 100644
--- a/sql/mysqld.cc
+++ b/sql/mysqld.cc
@@ -329,6 +329,7 @@ static char *mysqld_user, *mysqld_chroot, *log_error_file_ptr;
static char *opt_init_slave, *language_ptr, *opt_init_connect;
static char *default_character_set_name;
static char *character_set_filesystem_name;
+static char *lc_time_names_name;
static char *my_bind_addr_str;
static char *default_collation_name;
static char *default_storage_engine_str;
@@ -565,6 +566,8 @@ CHARSET_INFO *system_charset_info, *files_charset_info ;
CHARSET_INFO *national_charset_info, *table_alias_charset;
CHARSET_INFO *character_set_filesystem;
+MY_LOCALE *my_default_lc_time_names;
+
SHOW_COMP_OPTION have_ssl, have_symlink, have_dlopen, have_query_cache;
SHOW_COMP_OPTION have_geometry, have_rtree_keys;
SHOW_COMP_OPTION have_crypt, have_compress;
@@ -2913,6 +2916,14 @@ static int init_common_variables(const char *conf_file_name, int argc,
return 1;
global_system_variables.character_set_filesystem= character_set_filesystem;
+ if (!(my_default_lc_time_names=
+ my_locale_by_name(lc_time_names_name)))
+ {
+ sql_print_error("Unknown locale: '%s'", MYF(0), lc_time_names_name);
+ return 1;
+ }
+ global_system_variables.lc_time_names= my_default_lc_time_names;
+
sys_init_connect.value_length= 0;
if ((sys_init_connect.value= opt_init_connect))
sys_init_connect.value_length= strlen(opt_init_connect);
@@ -5011,6 +5022,7 @@ enum options_mysqld
OPT_DEFAULT_COLLATION,
OPT_CHARACTER_SET_CLIENT_HANDSHAKE,
OPT_CHARACTER_SET_FILESYSTEM,
+ OPT_LC_TIME_NAMES,
OPT_INIT_CONNECT,
OPT_INIT_SLAVE,
OPT_SECURE_AUTH,
@@ -5354,6 +5366,11 @@ Disable with --skip-innodb-doublewrite.", (gptr*) &innobase_use_doublewrite,
"Client error messages in given language. May be given as a full path.",
(gptr*) &language_ptr, (gptr*) &language_ptr, 0, GET_STR, REQUIRED_ARG,
0, 0, 0, 0, 0, 0},
+ {"lc-time-names", OPT_LC_TIME_NAMES,
+ "Set the language used for the month names and the days of the week.",
+ (gptr*) &lc_time_names_name,
+ (gptr*) &lc_time_names_name,
+ 0, GET_STR, REQUIRED_ARG, 0, 0, 0, 0, 0, 0 },
{"local-infile", OPT_LOCAL_INFILE,
"Enable/disable LOAD DATA LOCAL INFILE (takes values 1|0).",
(gptr*) &opt_local_infile,
@@ -7220,7 +7237,7 @@ static void mysql_init_variables(void)
default_collation_name= compiled_default_collation_name;
sys_charset_system.value= (char*) system_charset_info->csname;
character_set_filesystem_name= (char*) "binary";
-
+ lc_time_names_name= (char*) "en_US";
/* Set default values for some option variables */
default_storage_engine_str= (char*) "MyISAM";
global_system_variables.table_type= myisam_hton;
diff --git a/sql/set_var.cc b/sql/set_var.cc
index 7f55556c134..f7376a502fc 100644
--- a/sql/set_var.cc
+++ b/sql/set_var.cc
@@ -3038,7 +3038,10 @@ bool sys_var_thd_lc_time_names::check(THD *thd, set_var *var)
bool sys_var_thd_lc_time_names::update(THD *thd, set_var *var)
{
- thd->variables.lc_time_names= var->save_result.locale_value;
+ if (var->type == OPT_GLOBAL)
+ global_system_variables.lc_time_names= var->save_result.locale_value;
+ else
+ thd->variables.lc_time_names= var->save_result.locale_value;
return 0;
}
@@ -3046,13 +3049,18 @@ bool sys_var_thd_lc_time_names::update(THD *thd, set_var *var)
byte *sys_var_thd_lc_time_names::value_ptr(THD *thd, enum_var_type type,
LEX_STRING *base)
{
- return (byte *)(thd->variables.lc_time_names->name);
+ return type == OPT_GLOBAL ?
+ (byte *) global_system_variables.lc_time_names->name :
+ (byte *) thd->variables.lc_time_names->name;
}
void sys_var_thd_lc_time_names::set_default(THD *thd, enum_var_type type)
{
- thd->variables.lc_time_names = &my_locale_en_US;
+ if (type == OPT_GLOBAL)
+ global_system_variables.lc_time_names= my_default_lc_time_names;
+ else
+ thd->variables.lc_time_names= global_system_variables.lc_time_names;
}
/*
diff --git a/sql/sql_class.cc b/sql/sql_class.cc
index cc38d63c9f9..d4d8084b301 100644
--- a/sql/sql_class.cc
+++ b/sql/sql_class.cc
@@ -378,7 +378,6 @@ void THD::init(void)
update_charset();
reset_current_stmt_binlog_row_based();
bzero((char *) &status_var, sizeof(status_var));
- variables.lc_time_names = &my_locale_en_US;
}