MySQL Bugs: #41952: Please add a maria-transactional mysqld option

Bug #41952	Please add a maria-transactional mysqld option
Submitted:	8 Jan 2009 9:38	Modified:	26 May 2010 17:51
Reporter:	Philip Stoev	Email Updates:
Status:	Unsupported	Impact on me:	None
Category:	MySQL Server: Maria storage engine	Severity:	S2 (Serious)
Version:	6.0, 5.1-maria	OS:	Any
Assigned to:		CPU Architecture:	Any
Triage:	Triaged: D3 (Medium) / R2 (Low) / E3 (Medium)

Description:
Please add a --maria-transactional option to mysqld, so that the Maria mode of operation can be selected server-wide and not just per-table using CREATE TABLE TRANSACTIONAL.

This is very important in order to run the existing tests both with maria transactional=1 and transactional=0 without modifying all CREATE TABLE inside the tests. Otherwise, just the default transactional=1 is being tested and transactional=0 will not receive any coverage.

How to repeat:
Attempt to write a test that can run against Falcon, Innodb, PBXT , Maria transactional and maria non-transactional. No other engine uses CREATE TABLE options exclusively to modify basic engine behavoir.

Suggested fix:
Create a mysqld option --maria-transactional with default value of 1. If CREATE TABLE TRANSACTIONAL is not specified explicitly, the --maria-transactional option becomes effective.

It's a MySQL habit to do this, and I can see the value, so it will probably be approved, but it then has the usual problems: replication of this option (if master and slave are accidentally not using the same value of the option)...

http://lists2.mysql.com/maria/379

tried the patch and:
CURRENT_TEST: maria.maria
--- /home/mysql_src/bzrrepos/mysql-maria2/mysql-test/suite/maria/r/maria.result 2009-02-11 23:48:05.000000000 +0300
+++ /home/mysql_src/bzrrepos/mysql-maria2/mysql-test/suite/maria/r/maria.reject 2009-02-13 21:39:10.000000000 +0300
@@ -637,6 +637,8 @@
 Table  Non_unique      Key_name        Seq_in_index    Column_name     Collation       Cardinality     Sub_part        Packed  Null    Index_type      Comment
 t1     1       a       1       a       A       1000    NULL    NULL    YES     BTREE
 alter table t1 engine=heap;
+Warnings:
+Error  1478    Table storage engine 'MEMORY' does not support the create option 'TRANSACTIONAL=1'
 alter table t1 disable keys;
 Warnings:
 Note   1031    Table storage engine for 't1' doesn't have this option
@@ -690,14 +692,20 @@
 Table  Checksum
 test.t1        3235292310
 alter table t1 row_format=fixed;
+Warnings:
+Note   1478    Row format set to PAGE because of TRANSACTIONAL=1 option

Note the note: row_format_fixed is discarded.

Workaround is: specify TRANSACTIONAL=0|1 in CREATE TABLE.
Impact: a substantial amount of people could want to use this option.
Effort: Medium; Sanja's patch does not work which shows that it's not a Low effort. The inner workings of CREATE TABLE and SHOW CREATE TABLE are not so simple. 2 days of work?