From: gluh Date: April 19 2008 2:52pm Subject: bk commit into 6.0 tree (gluh:1.2620) BUG#5299 List-Archive: http://lists.mysql.com/commits/45655 X-Bug: 5299 Message-Id: <20080419145218.75C49D60240@mgluh.home> Below is the list of changes that have just been committed into a local 6.0 repository of gluh. When gluh does a push these changes will be propagated to the main repository and, within 24 hours after the push, to the public repository. For information on how to access the public repository see http://dev.mysql.com/doc/mysql/en/installing-source-tree.html ChangeSet@stripped, 2008-04-19 19:52:10+05:00, gluh@stripped +10 -0 Bug#5299 Remove SHOW COLUMN TYPES Removed 'SHOW COLUMN TYPES' command mysql-test/r/sp-code.result@stripped, 2008-04-19 19:52:07+05:00, gluh@stripped +2 -2 Removed 'SHOW COLUMN TYPES' command mysql-test/r/sp.result@stripped, 2008-04-19 19:52:07+05:00, gluh@stripped +0 -7 Removed 'SHOW COLUMN TYPES' command mysql-test/t/sp.test@stripped, 2008-04-19 19:52:07+05:00, gluh@stripped +0 -1 Removed 'SHOW COLUMN TYPES' command sql/mysqld.cc@stripped, 2008-04-19 19:52:07+05:00, gluh@stripped +0 -1 Removed 'SHOW COLUMN TYPES' command sql/sp_head.cc@stripped, 2008-04-19 19:52:07+05:00, gluh@stripped +0 -1 Removed 'SHOW COLUMN TYPES' command sql/sql_lex.h@stripped, 2008-04-19 19:52:07+05:00, gluh@stripped +1 -1 Removed 'SHOW COLUMN TYPES' command sql/sql_parse.cc@stripped, 2008-04-19 19:52:07+05:00, gluh@stripped +0 -4 Removed 'SHOW COLUMN TYPES' command sql/sql_prepare.cc@stripped, 2008-04-19 19:52:08+05:00, gluh@stripped +0 -1 Removed 'SHOW COLUMN TYPES' command sql/sql_show.cc@stripped, 2008-04-19 19:52:08+05:00, gluh@stripped +0 -94 Removed 'SHOW COLUMN TYPES' command sql/sql_yacc.yy@stripped, 2008-04-19 19:52:08+05:00, gluh@stripped +0 -5 Removed 'SHOW COLUMN TYPES' command diff -Nrup a/mysql-test/r/sp-code.result b/mysql-test/r/sp-code.result --- a/mysql-test/r/sp-code.result 2008-03-26 04:04:52 +04:00 +++ b/mysql-test/r/sp-code.result 2008-04-19 19:52:07 +05:00 @@ -155,11 +155,11 @@ Pos Instruction 0 stmt 9 "drop temporary table if exists sudoku..." 1 stmt 1 "create temporary table sudoku_work ( ..." 2 stmt 1 "create temporary table sudoku_schedul..." -3 stmt 90 "call sudoku_init()" +3 stmt 89 "call sudoku_init()" 4 jump_if_not 7(8) p_naive@0 5 stmt 4 "update sudoku_work set cnt = 0 where ..." 6 jump 8 -7 stmt 90 "call sudoku_count()" +7 stmt 89 "call sudoku_count()" 8 stmt 6 "insert into sudoku_schedule (row,col)..." 9 set v_scounter@2 0 10 set v_i@3 1 diff -Nrup a/mysql-test/r/sp.result b/mysql-test/r/sp.result --- a/mysql-test/r/sp.result 2008-04-01 20:19:11 +05:00 +++ b/mysql-test/r/sp.result 2008-04-19 19:52:07 +05:00 @@ -2438,7 +2438,6 @@ create procedure bug4902() begin show charset like 'foo'; show collation like 'foo'; -show column types; show create table t1; show create database test; show databases like 'foo'; @@ -2455,9 +2454,6 @@ end| call bug4902()| Charset Description Default collation Maxlen Collation Charset Id Default Compiled Sortlen -Type Size Min_Value Max_Value Prec Scale Nullable Auto_Increment Unsigned Zerofill Searchable Case_Sensitive Default Comment Storage Format -tinyint 1 -128 127 0 0 YES YES NO YES YES NO NULL,0 A very small integer Default Default -tinyint unsigned 1 0 255 0 0 YES YES YES YES YES NO NULL,0 A very small integer Default Default Table Create Table t1 CREATE TABLE `t1` ( `id` char(16) NOT NULL DEFAULT '', @@ -2509,9 +2505,6 @@ Level Code Message call bug4902()| Charset Description Default collation Maxlen Collation Charset Id Default Compiled Sortlen -Type Size Min_Value Max_Value Prec Scale Nullable Auto_Increment Unsigned Zerofill Searchable Case_Sensitive Default Comment Storage Format -tinyint 1 -128 127 0 0 YES YES NO YES YES NO NULL,0 A very small integer Default Default -tinyint unsigned 1 0 255 0 0 YES YES YES YES YES NO NULL,0 A very small integer Default Default Table Create Table t1 CREATE TABLE `t1` ( `id` char(16) NOT NULL DEFAULT '', diff -Nrup a/mysql-test/t/sp.test b/mysql-test/t/sp.test --- a/mysql-test/t/sp.test 2008-03-27 13:56:02 +04:00 +++ b/mysql-test/t/sp.test 2008-04-19 19:52:07 +05:00 @@ -2937,7 +2937,6 @@ create procedure bug4902() begin show charset like 'foo'; show collation like 'foo'; - show column types; show create table t1; show create database test; show databases like 'foo'; diff -Nrup a/sql/mysqld.cc b/sql/mysqld.cc --- a/sql/mysqld.cc 2008-04-16 00:45:51 +05:00 +++ b/sql/mysqld.cc 2008-04-19 19:52:07 +05:00 @@ -3178,7 +3178,6 @@ SHOW_VAR com_status_vars[]= { {"show_binlogs", (char*) offsetof(STATUS_VAR, com_stat[(uint) SQLCOM_SHOW_BINLOGS]), SHOW_LONG_STATUS}, {"show_charsets", (char*) offsetof(STATUS_VAR, com_stat[(uint) SQLCOM_SHOW_CHARSETS]), SHOW_LONG_STATUS}, {"show_collations", (char*) offsetof(STATUS_VAR, com_stat[(uint) SQLCOM_SHOW_COLLATIONS]), SHOW_LONG_STATUS}, - {"show_column_types", (char*) offsetof(STATUS_VAR, com_stat[(uint) SQLCOM_SHOW_COLUMN_TYPES]), SHOW_LONG_STATUS}, {"show_contributors", (char*) offsetof(STATUS_VAR, com_stat[(uint) SQLCOM_SHOW_CONTRIBUTORS]), SHOW_LONG_STATUS}, {"show_create_db", (char*) offsetof(STATUS_VAR, com_stat[(uint) SQLCOM_SHOW_CREATE_DB]), SHOW_LONG_STATUS}, {"show_create_event", (char*) offsetof(STATUS_VAR, com_stat[(uint) SQLCOM_SHOW_CREATE_EVENT]), SHOW_LONG_STATUS}, diff -Nrup a/sql/sp_head.cc b/sql/sp_head.cc --- a/sql/sp_head.cc 2008-03-14 21:30:08 +04:00 +++ b/sql/sp_head.cc 2008-04-19 19:52:07 +05:00 @@ -173,7 +173,6 @@ sp_get_flags_for_command(LEX *lex) case SQLCOM_SHOW_BINLOG_EVENTS: case SQLCOM_SHOW_CHARSETS: case SQLCOM_SHOW_COLLATIONS: - case SQLCOM_SHOW_COLUMN_TYPES: case SQLCOM_SHOW_CONTRIBUTORS: case SQLCOM_SHOW_CREATE: case SQLCOM_SHOW_CREATE_DB: diff -Nrup a/sql/sql_lex.h b/sql/sql_lex.h --- a/sql/sql_lex.h 2008-04-01 18:44:54 +05:00 +++ b/sql/sql_lex.h 2008-04-19 19:52:07 +05:00 @@ -94,7 +94,7 @@ enum enum_sql_command { SQLCOM_SHOW_SLAVE_HOSTS, SQLCOM_DELETE_MULTI, SQLCOM_UPDATE_MULTI, SQLCOM_SHOW_BINLOG_EVENTS, SQLCOM_SHOW_NEW_MASTER, SQLCOM_DO, SQLCOM_SHOW_WARNS, SQLCOM_EMPTY_QUERY, SQLCOM_SHOW_ERRORS, - SQLCOM_SHOW_COLUMN_TYPES, SQLCOM_SHOW_STORAGE_ENGINES, SQLCOM_SHOW_PRIVILEGES, + SQLCOM_SHOW_STORAGE_ENGINES, SQLCOM_SHOW_PRIVILEGES, SQLCOM_HELP, SQLCOM_CREATE_USER, SQLCOM_DROP_USER, SQLCOM_RENAME_USER, SQLCOM_REVOKE_ALL, SQLCOM_CHECKSUM, SQLCOM_CREATE_PROCEDURE, SQLCOM_CREATE_SPFUNCTION, SQLCOM_CALL, diff -Nrup a/sql/sql_parse.cc b/sql/sql_parse.cc --- a/sql/sql_parse.cc 2008-04-16 00:45:51 +05:00 +++ b/sql/sql_parse.cc 2008-04-19 19:52:07 +05:00 @@ -252,7 +252,6 @@ void init_update_queries(void) sql_command_flags[SQLCOM_SHOW_BINLOGS]= CF_STATUS_COMMAND; sql_command_flags[SQLCOM_SHOW_SLAVE_HOSTS]= CF_STATUS_COMMAND; sql_command_flags[SQLCOM_SHOW_BINLOG_EVENTS]= CF_STATUS_COMMAND; - sql_command_flags[SQLCOM_SHOW_COLUMN_TYPES]= CF_STATUS_COMMAND; sql_command_flags[SQLCOM_SHOW_STORAGE_ENGINES]= CF_STATUS_COMMAND; sql_command_flags[SQLCOM_SHOW_AUTHORS]= CF_STATUS_COMMAND; sql_command_flags[SQLCOM_SHOW_CONTRIBUTORS]= CF_STATUS_COMMAND; @@ -3099,9 +3098,6 @@ end_with_restore_list: break; case SQLCOM_SHOW_PRIVILEGES: res= mysqld_show_privileges(thd); - break; - case SQLCOM_SHOW_COLUMN_TYPES: - res= mysqld_show_column_types(thd); break; case SQLCOM_SHOW_ENGINE_LOGS: #ifdef DONT_ALLOW_SHOW_COMMANDS diff -Nrup a/sql/sql_prepare.cc b/sql/sql_prepare.cc --- a/sql/sql_prepare.cc 2008-03-28 18:49:09 +04:00 +++ b/sql/sql_prepare.cc 2008-04-19 19:52:08 +05:00 @@ -1800,7 +1800,6 @@ static bool check_prepared_statement(Pre case SQLCOM_SHOW_PROCESSLIST: case SQLCOM_SHOW_STORAGE_ENGINES: case SQLCOM_SHOW_PRIVILEGES: - case SQLCOM_SHOW_COLUMN_TYPES: case SQLCOM_SHOW_ENGINE_LOGS: case SQLCOM_SHOW_ENGINE_STATUS: case SQLCOM_SHOW_ENGINE_MUTEX: diff -Nrup a/sql/sql_show.cc b/sql/sql_show.cc --- a/sql/sql_show.cc 2008-04-19 00:00:26 +05:00 +++ b/sql/sql_show.cc 2008-04-19 19:52:08 +05:00 @@ -340,100 +340,6 @@ bool mysqld_show_privileges(THD *thd) } -/*************************************************************************** - List all column types -***************************************************************************/ - -struct show_column_type_st -{ - const char *type; - uint size; - const char *min_value; - const char *max_value; - uint precision; - uint scale; - const char *nullable; - const char *auto_increment; - const char *unsigned_attr; - const char *zerofill; - const char *searchable; - const char *case_sensitivity; - const char *default_value; - const char *comment; - const char *storage_type; - const char *column_format; -}; - -/* TODO: Add remaning types */ - -static struct show_column_type_st sys_column_types[]= -{ - {"tinyint", - 1, "-128", "127", 0, 0, "YES", "YES", - "NO", "YES", "YES", "NO", "NULL,0", - "A very small integer", "Default", "Default"}, - {"tinyint unsigned", - 1, "0" , "255", 0, 0, "YES", "YES", - "YES", "YES", "YES", "NO", "NULL,0", - "A very small integer", "Default", "Default"}, -}; - -bool mysqld_show_column_types(THD *thd) -{ - List field_list; - Protocol *protocol= thd->protocol; - DBUG_ENTER("mysqld_show_column_types"); - - field_list.push_back(new Item_empty_string("Type",30)); - field_list.push_back(new Item_int("Size",(longlong) 1, - MY_INT64_NUM_DECIMAL_DIGITS)); - field_list.push_back(new Item_empty_string("Min_Value",20)); - field_list.push_back(new Item_empty_string("Max_Value",20)); - field_list.push_back(new Item_return_int("Prec", 4, MYSQL_TYPE_SHORT)); - field_list.push_back(new Item_return_int("Scale", 4, MYSQL_TYPE_SHORT)); - field_list.push_back(new Item_empty_string("Nullable",4)); - field_list.push_back(new Item_empty_string("Auto_Increment",4)); - field_list.push_back(new Item_empty_string("Unsigned",4)); - field_list.push_back(new Item_empty_string("Zerofill",4)); - field_list.push_back(new Item_empty_string("Searchable",4)); - field_list.push_back(new Item_empty_string("Case_Sensitive",4)); - field_list.push_back(new Item_empty_string("Default",NAME_CHAR_LEN)); - field_list.push_back(new Item_empty_string("Comment",NAME_CHAR_LEN)); - field_list.push_back(new Item_empty_string("Storage",8)); - field_list.push_back(new Item_empty_string("Format",8)); - - if (protocol->send_fields(&field_list, - Protocol::SEND_NUM_ROWS | Protocol::SEND_EOF)) - DBUG_RETURN(TRUE); - - /* TODO: Change the loop to not use 'i' */ - for (uint i=0; i < sizeof(sys_column_types)/sizeof(sys_column_types[0]); i++) - { - protocol->prepare_for_resend(); - protocol->store(sys_column_types[i].type, system_charset_info); - protocol->store((ulonglong) sys_column_types[i].size); - protocol->store(sys_column_types[i].min_value, system_charset_info); - protocol->store(sys_column_types[i].max_value, system_charset_info); - protocol->store_short((longlong) sys_column_types[i].precision); - protocol->store_short((longlong) sys_column_types[i].scale); - protocol->store(sys_column_types[i].nullable, system_charset_info); - protocol->store(sys_column_types[i].auto_increment, system_charset_info); - protocol->store(sys_column_types[i].unsigned_attr, system_charset_info); - protocol->store(sys_column_types[i].zerofill, system_charset_info); - protocol->store(sys_column_types[i].searchable, system_charset_info); - protocol->store(sys_column_types[i].case_sensitivity, system_charset_info); - protocol->store(sys_column_types[i].default_value, system_charset_info); - protocol->store(sys_column_types[i].comment, system_charset_info); - protocol->store(sys_column_types[i].storage_type, system_charset_info); - protocol->store(sys_column_types[i].column_format, system_charset_info); - if (protocol->write()) - DBUG_RETURN(TRUE); - } - my_eof(thd); - DBUG_RETURN(FALSE); -} - - /* find_files() - find files in a given directory. diff -Nrup a/sql/sql_yacc.yy b/sql/sql_yacc.yy --- a/sql/sql_yacc.yy 2008-04-19 19:11:43 +05:00 +++ b/sql/sql_yacc.yy 2008-04-19 19:52:08 +05:00 @@ -9646,11 +9646,6 @@ show_param: if (prepare_schema_table(YYTHD, lex, $3, SCH_STATISTICS)) MYSQL_YYABORT; } - | COLUMN_SYM TYPES_SYM - { - LEX *lex=Lex; - lex->sql_command= SQLCOM_SHOW_COLUMN_TYPES; - } | opt_storage ENGINES_SYM { LEX *lex=Lex;