Bug #30088 Can't disable myisam-recover by a value of "".
Submitted: 27 Jul 2007 4:39 Modified: 3 Aug 2007 18:46
Reporter: Sadao Hiratsuka (Basic Quality Contributor)
Status: Closed
Category:Server: MyISAM Severity:S3 (Non-critical)
Version:5.0.42, 5.0.44 OS:Any
Assigned to: Ramil Kalimullin Target Version:
Tags: myisam-recover, myisam, qc

[27 Jul 2007 4: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 5:08] Valeriy Kravchuk 
Thank you for a bug report. Verified just as described.
[1 Aug 2007 8: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 12:20] Sergey Gluhov 
ok to push
[2 Aug 2007 21:13] Bugs System 
Pushed into 5.1.21-beta
[2 Aug 2007 21:16] Bugs System 
Pushed into 5.0.48
[3 Aug 2007 18:46] Paul DuBois 
Noted in 5.0.48, 5.1.21 changelogs.

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