Bug #30088 Can't disable myisam-recover by a value of "".
Submitted: 27 Jul 2007 2:39 Modified: 3 Aug 2007 16:46
Reporter: Sadao Hiratsuka (Basic Quality Contributor) Email Updates:
Status: Closed Impact on me:
None 
Category:MySQL Server: MyISAM storage engine Severity:S3 (Non-critical)
Version:5.0.42, 5.0.44 OS:Any
Assigned to: Ramil Kalimullin CPU Architecture:Any
Tags: myisam, myisam-recover, qc

[27 Jul 2007 2:39] Sadao Hiratsuka 
Description:
I can't disable myisam-recover option by a value of "".

See 5.2.2. Command Options
http://dev.mysql.com/doc/refman/5.0/en/server-options.html
--myisam-recover[=option[,option]...]]
Set the MyISAM storage engine recovery mode. The option value is any combination of the values of DEFAULT, BACKUP, FORCE, or QUICK. If you specify multiple values, separate them by commas. You can also use a value of "" to disable this option.

How to repeat:
(1) with no option in my.cnf
root:(none)> show variables like 'myisam_recover_options';
+------------------------+-------+
| Variable_name          | Value |
+------------------------+-------+
| myisam_recover_options | OFF   |
+------------------------+-------+
1 row in set (0.00 sec)

(2) myisam-recover = ""
root:(none)> show variables like 'myisam_recover_options';
+------------------------+---------+
| Variable_name          | Value   |
+------------------------+---------+
| myisam_recover_options | DEFAULT | *** I think OFF is preferable.
+------------------------+---------+
1 row in set (0.00 sec)

(3) myisam-recover = DEFAULT
root:(none)> show variables like 'myisam_recover_options';
+------------------------+---------+
| Variable_name          | Value   |
+------------------------+---------+
| myisam_recover_options | DEFAULT |
+------------------------+---------+
1 row in set (0.00 sec)

Suggested fix:
For example, add NONE option explicitly,
or modify Reference Manual.
[27 Jul 2007 3:08] Valeriy Kravchuk 
Thank you for a bug report. Verified just as described.
[1 Aug 2007 6:41] Bugs System 
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/31934

ChangeSet@1.2482, 2007-08-01 11:41:13+05:00, ramil@mysql.com +4 -0
  Fix for bug #30088: Can't disable myisam-recover by a value of "".
    - test result.
  
  Problem: we don't proper handle "" value of the --myisam-recover option.
  
  Fix: turn off myisam recovering if --myisam-recover="" is set.
[2 Aug 2007 10:20] Sergei Glukhov 
ok to push
[2 Aug 2007 19:13] Bugs System 
Pushed into 5.1.21-beta
[2 Aug 2007 19:16] Bugs System 
Pushed into 5.0.48
[3 Aug 2007 16:46] Paul DuBois 
Noted in 5.0.48, 5.1.21 changelogs.

--myisam-recover="" (empty option value) did not disable MyISAM
recovery.