[MDEV-392] MTR: skip-combinations option is declared in help, but is ignored Created: 2012-07-12  Updated: 2012-08-08  Resolved: 2012-08-08

Status: Closed
Project: MariaDB Server
Component/s: None
Affects Version/s: 5.5.25
Fix Version/s: 5.5.27

Type: Bug Priority: Trivial
Reporter: Elena Stepanova Assignee: Sergei Golubchik
Resolution: Fixed Votes: 0
Labels: tests


 Description   

Before 5.5 (or possibly before MDEV-30), skip-combinations provided on MTR command line worked as the name suggests. In 5.5 it is declared and accepted, but ignored (and in the code seems to be no use of it, instead there is a hash which is read from the suite file).

MariaDB 5.3:

 
 
ubuntu11:~/maria-5.3/mysql-test$ perl ./mtr binlog.binlog_tmp_table --skip-combinations
 
 
 
==============================================================================
 
 
 
TEST                                      RESULT   TIME (ms)
 
------------------------------------------------------------
 
 
 
worker[1] Using MTR_BUILD_THREAD 300, with reserved ports 16000..16019
 
binlog.binlog_tmp_table                  [ pass ]   2055
 
------------------------------------------------------------
 
The servers were restarted 0 times
 
Spent 2.055 of 4 seconds executing testcases
 
 
 
Completed: All 1 tests were successful.

MariaDB 5.5 (revno 3462):

 
 
perl ./mtr binlog.binlog_tmp_table --skip-combinations
 
 
 
 
 
==============================================================================
 
 
 
TEST                                      RESULT   TIME (ms) or COMMENT
 
--------------------------------------------------------------------------
 
 
 
worker[1] Using MTR_BUILD_THREAD 300, with reserved ports 16000..16019
 
binlog.binlog_tmp_table 'mix'            [ pass ]    326
 
binlog.binlog_tmp_table 'row'            [ skipped ]  Neither MIXED nor STATEMENT binlog format
 
binlog.binlog_tmp_table 'stmt'           [ pass ]    323
 
--------------------------------------------------------------------------
 
The servers were restarted 2 times
 
Spent 0.649 of 3 seconds executing testcases
 
 
 
Completed: All 2 tests were successful.
 
 
 
1 tests were skipped, 1 by the test itself.

It's a good option, would be nice to have it back if possible.

 Comments   
Comment by Sergei Golubchik [ 2012-07-12 ]

I've intentionally removed it in MDEV-30. In MySQL combinations were, more or less, optional. One could imagine running tests without them. After MDEV-30 they're essential and infuence heavily what tests are run and how. If combinations were completely ignored, the server, probably, wouldn't even start.

What exactly do you want to achieve with --skip-combinations option?

Comment by Elena Stepanova [ 2012-07-12 ]

I expected it to be as simple as completely ignoring the combinations file and let the user or fate decide whether it contained mandatory parameters or optional ones. Like in the example above, ignore 3 binlog formats and run only once, with whatever server or MTR sets by default.

That said, I don't insist on it, since it can be achieved by other means (hence the 'Trivial' priority). If it was intentional, lets just remove the option from usage printed by MTR, and from the list of options it reads and saves. Then, if somebody puts the option on the command line, it will be passed through to the server, and server will die of ignorance – which is also a bit confusing, but not as much as a declared but non-functional option, and the behavior will be consistent with other skip-<don't know what> options.

Generated at Thu Feb 08 06:28:24 UTC 2024 using Jira 8.20.16#820016-sha1:9d11dbea5f4be3d4cc21f03a88dd11d8c8687422.