Default file name changed to "/etc/default/mysql" ; The MYCHECK_RCPT variable can now be set from the default file ; The check_for_crashed_tables() function on the debian-start script has been fixed and now also checks Aria tables

Author: jb-boin

debian/additions/debian-start

@@ -13,9 +13,9 @@ MYUPGRADE="/usr/bin/mysql_upgrade --defaults-extra-file=/etc/mysql/debian.cnf"
 MYCHECK="/usr/bin/mysqlcheck --defaults-file=/etc/mysql/debian.cnf"
 MYCHECK_SUBJECT="WARNING: mysqlcheck has found corrupt tables"
 MYCHECK_PARAMS="--all-databases --fast --silent"
-MYCHECK_RCPT="root"
+MYCHECK_RCPT="${MYCHECK_RCPT:-root}"
 
-## Checking for corrupt, not cleanly closed and upgrade needing tables.
+## Checking for corrupt, not cleanly closed (only for MyISAM and Aria engines) and upgrade needing tables.
 
 # The following commands should be run when the server is up but in background
 # where they do not block the server start and in one shell instance so that

debian/additions/debian-start.inc.sh

@@ -3,15 +3,15 @@
 # This file is included by /etc/mysql/debian-start
 #
 
-## Check all unclosed tables.
+## Check MyISAM and Aria unclosed tables.
 # - Requires the server to be up.
 # - Is supposed to run silently in background. 
 function check_for_crashed_tables() {
   set -e
   set -u
 
   # But do it in the background to not stall the boot process.
-  logger -p daemon.info -i -t$0 "Triggering myisam-recover for all MyISAM tables"
+  logger -p daemon.info -i -t$0 "Triggering myisam-recover for all MyISAM tables and aria-recover for all Aria tables"
 
   # Checking for $? is unreliable so the size of the output is checked.
   # Some table handlers like HEAP do not support CHECK TABLE.
@@ -21,10 +21,10 @@ function check_for_crashed_tables() {
   LC_ALL=C $MYSQL --skip-column-names --batch -e  '
       select concat('\''select count(*) into @discard from `'\'',
                     TABLE_SCHEMA, '\''`.`'\'', TABLE_NAME, '\''`'\'') 
-      from information_schema.TABLES where ENGINE='\''MyISAM'\' | \
+      from information_schema.TABLES where TABLE_SCHEMA<>'\''INFORMATION_SCHEMA'\'' and TABLE_SCHEMA<>'\''PERFORMANCE_SCHEMA'\'' and ( ENGINE='\''MyISAM'\'' or ENGINE='\''Aria'\'' )' | \
     xargs -i $MYSQL --skip-column-names --silent --batch \
-                    --force -e "{}" >$tempfile 
-  if [ -s $tempfile ]; then
+                    --force -e "{}" &>$tempfile 
+  if [ -s "$tempfile" ]; then
     (
       /bin/echo -e "\n" \
         "Improperly closed tables are also reported if clients are accessing\n" \

debian/changelog

@@ -31,6 +31,11 @@ mariadb-10.0 (10.0.20-3) unstable; urgency=medium
   [ Jean Weisbuch ]
   * Added a Debian default file for the mariadb-server-10.0 package which allows
     one to set the MYSQLD_STARTUP_TIMEOUT variable used in the init script
+  * The MYCHECK_RCPT variable can now be set from the default file.
+  * The check_for_crashed_tables() function on the debian-start script has been 
+    fixed to be able to log (and email) the errors it encountered : Errors are
+    sent to stderr by the CLI while only stdout was captured by the function.
+  * The same function now also checks Aria tables along with MyISAM ones. 
 
  -- Otto Kekäläinen <[email protected]>  Fri, 24 Jul 2015 23:00:00 +0300
 

debian/mariadb-server-10.0.default debian/mariadb-server-10.0.mysql.default

@@ -2,3 +2,7 @@
 # If the server is still not responding after the delay, the script won't be executed and an error will be thrown on the syslog.
 # Default: 30
 #MYSQLD_STARTUP_TIMEOUT=30
+
+# The email recipient(s) of the output of the check for crashed and improperly closed MyISAM and Aria tables done at each server start by the "/etc/mysql/debian-start" script.
+# Default: root
+#MYCHECK_RCPT="root"