Split error message for crashed table to 3 different error messages.

Fixes for Windows


Docs/manual.texi:
  Added things to the Windows section
client/mysql.cc:
  Fixes to get Jani's changes to work on Windows
include/myisam.h:
  Added chk_status()
include/mysqld_error.h:
  Split error message for crashed table to 3 different error messages
myisam/mi_check.c:
  Added chk_status()
myisam/myisamchk.c:
  Added chk_status(). This gives us better error messages when the
  table is marked as 'crashed'
sql/ha_myisam.cc:
  Better error messages if the table is marked as crashed
sql/handler.cc:
  Better error messages if the table is marked as crashed
sql/share/czech/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/czech/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/danish/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/danish/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/dutch/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/dutch/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/english/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/english/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/estonian/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/estonian/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/french/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/french/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/german/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/german/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/greek/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/greek/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/hungarian/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/hungarian/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/italian/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/italian/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/japanese/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/japanese/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/korean/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/korean/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/norwegian-ny/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/norwegian/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/polish/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/polish/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/portuguese/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/portuguese/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/romanian/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/russian/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/russian/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/slovak/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/slovak/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/spanish/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/spanish/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/share/swedish/errmsg.OLD:
  Better error messages if the table is marked as crashed
sql/share/swedish/errmsg.sys:
  Better error messages if the table is marked as crashed
sql/share/swedish/errmsg.txt:
  Better error messages if the table is marked as crashed
sql/sql_table.cc:
  Fix for gcc 2.7.2

Docs/manual.texi

@@ -8262,6 +8262,12 @@ following information:
 @example
 127.0.0.1       localhost
 @end example
+
+@item @code{ALTER TABLE}
+While you are doing an @code{ALTER TABLE} the table is locked from usage
+by other threads. This has to do with the fact that you on Windows can't
+delete a file that is in use by another threads. (We may in the future
+find some way to go around this problem).
 @end table
 
 Here are some open issues for anyone who might want to help us with the Windows

client/mysql.cc

@@ -145,14 +145,18 @@ void tee_putc(int c, FILE *file);
 static int get_options(int argc,char **argv);
 static int com_quit(String *str,char*),
 	   com_go(String *str,char*), com_ego(String *str,char*),
-	   com_edit(String *str,char*), com_print(String *str,char*),
+	   com_print(String *str,char*),
 	   com_help(String *str,char*), com_clear(String *str,char*),
 	   com_connect(String *str,char*), com_status(String *str,char*),
 	   com_use(String *str,char*), com_source(String *str, char*),
-	   com_rehash(String *str, char*), com_pager(String *str, char*),
-           com_nopager(String *str, char*), com_tee(String *str, char*),
+	   com_rehash(String *str, char*), com_tee(String *str, char*),
            com_notee(String *str, char*);
 
+#ifndef __WIN__
+static int com_nopager(String *str, char*), com_pager(String *str, char*),
+	   com_edit(String *str,char*);
+#endif
+
 static int read_lines(bool execute_commands);
 static int sql_connect(char *host,char *database,char *user,char *password,
 		       uint silent);
@@ -180,7 +184,9 @@ static COMMANDS commands[] = {
   { "clear",  'c', com_clear,  0, "Clear command."},
   { "connect",'r', com_connect,1,
     "Reconnect to the server. Optional arguments are db and host." },
+#ifndef __WIN__
   { "edit",   'e', com_edit,   0, "Edit command with $EDITOR."},
+#endif
   { "ego",    'G', com_ego,    0,
     "Send command to mysql server, display result vertically."},
   { "exit",   'q', com_quit,   0, "Exit mysql. Same as quit."},
@@ -764,6 +770,16 @@ static int read_lines(bool execute_commands)
     }
     else
     {
+#ifdef __WIN__
+      if (opt_outfile && glob_buffer.is_empty())
+	fflush(OUTFILE);
+      tee_fputs(glob_buffer.is_empty() ? "mysql> " :
+		!in_string ? "    -> " :
+		in_string == '\'' ?
+		"    '> " : "    \"> ",stdout);
+      linebuffer[0]=(char) sizeof(linebuffer);
+      line=_cgets(linebuffer);
+#else
       if (opt_outfile)
       {
 	if (glob_buffer.is_empty())
@@ -773,14 +789,6 @@ static int read_lines(bool execute_commands)
 	      in_string == '\'' ?
 	      "    '> " : "    \"> ", OUTFILE);
       }
-#ifdef __WIN__
-      tee_fprintf(stdout, glob_buffer.is_empty() ? "mysql> " :
-		  !in_string ? "    -> " :
-		  in_string == '\'' ?
-		  "    '> " : "    \"> ");
-      linebuffer[0]=(char) sizeof(linebuffer);
-      line=_cgets(linebuffer);
-#else
       line=readline((char*) (glob_buffer.is_empty() ? "mysql> " :
 			     !in_string ? "    -> " :
 			     in_string == '\'' ?
@@ -1476,7 +1484,7 @@ static void end_pager()
 
 static void init_tee()
 {
-  if (!(OUTFILE= my_fopen(outfile,O_APPEND | O_WRONLY | O_BINARY,MYF(MY_WME))))
+  if (!(OUTFILE= my_fopen(outfile, O_APPEND | O_WRONLY, MYF(MY_WME))))
   {
     opt_outfile=0;
     init_pager();
@@ -1727,7 +1735,7 @@ com_tee(String *buffer, char *line __attribute__((unused)))
     init_tee();
     opt_outfile=1;
   }
-  tee_fprintf(stdout, "Outfile '%s' is in use now.\n", outfile);
+  tee_fprintf(stdout, "Logging to file '%s'\n", outfile);
   return 0;
 }
 
@@ -1742,6 +1750,11 @@ com_notee(String *buffer __attribute__((unused)),
   return 0;
 }
 
+/*
+** Sorry, this command is not available in Windows.
+*/
+
+#ifndef __WIN__
 static int
 com_pager(String *buffer, char *line __attribute__((unused)))
 {
@@ -1749,10 +1762,6 @@ com_pager(String *buffer, char *line __attribute__((unused)))
 
   if (status.batch)
     return 0;
-#ifdef __WIN__
-  tee_fprintf(stdout, "Sorry, this command is not available in Windows.\n");
-  return 0;
-#endif
   /* Skip space from file name */
   while (isspace(*line))
     line++;
@@ -1783,28 +1792,27 @@ com_pager(String *buffer, char *line __attribute__((unused)))
   return 0;
 }
 
+
 static int
 com_nopager(String *buffer __attribute__((unused)),
 	    char *line __attribute__((unused)))
 {
-#ifdef __WIN__
-  tee_fprintf(stdout, "This command has no function in Windows.\n");
-  return 0;
-#endif
   strmov(pager, "stdout");
   opt_nopager=1;
   tee_fprintf(stdout, "PAGER set to stdout\n");
   return 0;
 }
+#endif
 
 
+/*
+** Sorry, you can't send the result to an editor in Win32
+*/
+
+#ifndef __WIN__
 static int
 com_edit(String *buffer,char *line __attribute__((unused)))
 {
-#ifdef __WIN__
-  put_info("Sorry, you can't send the result to an editor in Win32",
-	   INFO_ERROR);
-#else
   char	filename[FN_REFLEN],buff[160];
   int	fd,tmp;
   const char *editor;
@@ -1838,9 +1846,10 @@ com_edit(String *buffer,char *line __attribute__((unused)))
   (void) my_close(fd,MYF(0));
   (void) my_delete(filename,MYF(MY_WME));
 err:
-#endif
   return 0;
 }
+#endif
+
 
 /* If arg is given, exit without errors. This happens on command 'quit' */
 

include/myisam.h

@@ -353,6 +353,7 @@ typedef struct st_mi_sortinfo {
 
 /* functions in mi_check */
 void myisamchk_init(MI_CHECK *param);
+int chk_status(MI_CHECK *param, MI_INFO *info);
 int chk_del(MI_CHECK *param, register MI_INFO *info, uint test_flag);
 int chk_size(MI_CHECK *param, MI_INFO *info);
 int chk_key(MI_CHECK *param, MI_INFO *info);

include/mysqld_error.h

@@ -194,4 +194,6 @@
 #define ER_FT_MATCHING_KEY_NOT_FOUND 1191
 #define ER_LOCK_OR_ACTIVE_TRANSACTION 1192
 #define ER_UNKNOWN_SYSTEM_VARIABLE 1193
-#define ER_ERROR_MESSAGES 194
+#define ER_CRASHED_ON_USAGE 1194
+#define ER_CRASHED_ON_REPAIR 1195
+#define ER_ERROR_MESSAGES 196

myisam/mi_check.c

@@ -97,6 +97,26 @@ void myisamchk_init(MI_CHECK *param)
   param->start_check_pos=0;
 }
 
+	/* Check the status flags for the table */
+
+int chk_status(MI_CHECK *param, register MI_INFO *info)
+{
+  MYISAM_SHARE *share=info->s;
+  if (mi_is_crashed_on_repair(info))
+    mi_check_print_warning(param,
+			   "Table is marked as crashed and last repair failed");
+  else if (mi_is_crashed(info))
+    mi_check_print_warning(param,
+			   "Table is marked as crashed");
+  if (share->state.open_count)
+  {
+    mi_check_print_warning(param,
+			   "%d clients is using or hasn't closed the table properly",
+			   share->state.open_count);
+  }
+  return 0;
+}
+
 	/* Check delete links */
 
 int chk_del(MI_CHECK *param, register MI_INFO *info, uint test_flag)

myisam/myisamchk.c

@@ -196,7 +196,7 @@ static struct option long_options[] =
 
 static void print_version(void)
 {
-  printf("%s  Ver 1.37 for %s at %s\n",my_progname,SYSTEM_TYPE,
+  printf("%s  Ver 1.38 for %s at %s\n",my_progname,SYSTEM_TYPE,
 	 MACHINE_TYPE);
 }
 
@@ -739,12 +739,7 @@ static int myisamchk(MI_CHECK *param, my_string filename)
 	printf("Data records: %7s   Deleted blocks: %7s\n",
 	       llstr(info->state->records,llbuff),
 	       llstr(info->state->del,llbuff2));
-      if (share->state.open_count)
-      {
-	mi_check_print_warning(param,
-			       "%d clients is using or hasn't closed the table properly",
-			       share->state.open_count);
-      }
+      error =chk_status(param,info);
       share->state.key_map &=param->keys_in_use;
       error =chk_size(param,info);
       if (!error || !(param->testflag & (T_FAST | T_FORCE_CREATE)))

sql/ha_myisam.cc

@@ -259,6 +259,7 @@ int ha_myisam::check(THD* thd, HA_CHECK_OPT* check_opt)
 				      (uint) (share->global_changed ? 1 : 0)))))
     return HA_ADMIN_ALREADY_DONE;
 
+  error = chk_status(&param, file);		// Not fatal
   error = chk_size(&param, file);
   if (!error)
     error |= chk_del(&param, file, param.testflag);
@@ -266,8 +267,10 @@ int ha_myisam::check(THD* thd, HA_CHECK_OPT* check_opt)
     error = chk_key(&param, file);
   if (!error)
   {
-    if (!check_opt->quick &&
-	(share->options & (HA_OPTION_PACK_RECORD | HA_OPTION_COMPRESS_RECORD)))
+    if ((!check_opt->quick &&
+	(share->options &
+	 (HA_OPTION_PACK_RECORD | HA_OPTION_COMPRESS_RECORD))) ||
+	mi_is_crashed(file))
     {
       init_io_cache(&param.read_cache, file->dfile,
 		    my_default_record_cache_size, READ_CACHE,
@@ -281,7 +284,8 @@ int ha_myisam::check(THD* thd, HA_CHECK_OPT* check_opt)
     if ((share->state.changed & (STATE_CHANGED |
 				 STATE_CRASHED_ON_REPAIR |
 				 STATE_CRASHED | STATE_NOT_ANALYZED)) ||
-	(param.testflag & T_STATISTICS))
+	(param.testflag & T_STATISTICS) ||
+	mi_is_crashed(file))
     {
       file->update|=HA_STATE_CHANGED | HA_STATE_ROW_CHANGED;
       pthread_mutex_lock(&share->intern_lock);

sql/handler.cc

@@ -543,9 +543,14 @@ void handler::print_error(int error, myf errflag)
     textno=ER_CHECKREAD;
     break;
   case HA_ERR_CRASHED:
-  case HA_ERR_CRASHED_ON_REPAIR:
     textno=ER_NOT_KEYFILE;
     break;
+  case HA_ERR_CRASHED_ON_USAGE:
+    textno=ER_CRASHED_ON_USAGE;
+    break;
+  case HA_ERR_CRASHED_ON_REPAIR:
+    textno=ER_CRASHED_ON_REPAIR;
+    break;
   case HA_ERR_OUT_OF_MEM:
     my_error(ER_OUT_OF_RESOURCES,errflag);
     DBUG_VOID_RETURN;

sql/share/czech/errmsg.txt

@@ -209,3 +209,5 @@
 "-Bdn sloupec nem vytvoen fulltextov index",-A
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/danish/errmsg.txt

@@ -198,3 +198,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/dutch/errmsg.txt

@@ -195,3 +195,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/english/errmsg.txt

@@ -195,3 +195,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/estonian/errmsg.txt

@@ -199,3 +199,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/french/errmsg.txt

@@ -195,3 +195,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/german/errmsg.txt

@@ -198,3 +198,5 @@
 "Kann keinen FULLTEXT-Index finden der der Spaltenliste entspricht",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/greek/errmsg.txt

@@ -195,3 +195,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/hungarian/errmsg.txt

@@ -197,3 +197,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/italian/errmsg.txt

@@ -195,3 +195,5 @@
 "Impossibile trovare un indice FULLTEXT che corrisponda all'elenco delle colonne",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/japanese/errmsg.txt

@@ -197,3 +197,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/korean/errmsg.txt

@@ -195,3 +195,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/norwegian-ny/errmsg.txt

@@ -197,3 +197,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/norwegian/errmsg.txt

@@ -197,3 +197,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/polish/errmsg.txt

@@ -199,3 +199,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/portuguese/errmsg.txt

@@ -195,3 +195,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/romanian/errmsg.txt

@@ -199,3 +199,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/russian/errmsg.txt

@@ -198,3 +198,5 @@
 "FULLTEXT ,    ,  ",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/slovak/errmsg.txt

@@ -203,3 +203,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/spanish/errmsg.txt

@@ -196,3 +196,5 @@
 "Can't find FULLTEXT index matching the column list",
 "Can't execute the given command because you have active locked tables or an active transaction",
 "Unknown system variable '%-.64'",
+"Table '%-.64s' is marked as crashed and should be repaired",
+"Table '%-.64s' is marked as crashed and last (automatic?) repair failed",

sql/share/swedish/errmsg.OLD

@@ -195,3 +195,7 @@
 "Hittar inte ett FULLTEXT index i kolumnlistan",
 "Kan inte exekvera kommandot emedan du har en låst tabell eller an aktiv transaktion",
 "Okänd system variabel '%-.64'",
+#ER_CRASHED_ON_USAGE
+"Tabell '%-.64s' är crashad och bör repareras med REPAIR TABLE",
+#ER_CRASHED_ON_REPAIR
+"Tabell '%-.64s' är crashad och senast (automatiska?) reparation misslyckades",

sql/share/swedish/errmsg.txt

@@ -195,3 +195,5 @@
 "Hittar inte ett FULLTEXT index i kolumnlistan",
 "Kan inte exekvera kommandot emedan du har en låst tabell eller an aktiv transaktion",
 "Okänd system variabel '%-.64'",
+"Tabell '%-.64s' är crashad och bör repareras med REPAIR TABLE",
+"Tabell '%-.64s' är crashad och senast (automatiska?) reparation misslyckades",

sql/sql_table.cc

@@ -650,8 +650,8 @@ TABLE *create_table_from_items(THD *thd, HA_CREATE_INFO *create_info,
 				  (Item_result_field***) 0, &tmp_field,0,0);
     if (!field ||
 	!(cr_field=new create_field(field,(item->type() == Item::FIELD_ITEM ?
-					   ((Item_field *)item)->field : NULL)
-				    )))
+					   ((Item_field *)item)->field :
+					   (Field*) 0))))
       DBUG_RETURN(0);
     extra_fields->push_back(cr_field);
   }