← Back to team overview

maria-developers team mailing list archive

Re: [Commits] 828a7b6: MDEV-6858: enforce_storage_engine option

 

On Wed, Mar 11, 2015 at 10:10 AM, Sergei Golubchik <serg@xxxxxxxxxxx> wrote:
>> diff --git a/mysql-test/t/enforce_storage_engine.test b/mysql-test/t/enforce_storage_engine.test
>> new file mode 100644
>> index 0000000..80839e2
>> --- /dev/null
>> +++ b/mysql-test/t/enforce_storage_engine.test
>> @@ -0,0 +1,64 @@
>> +-- source include/have_innodb.inc
>
> You can keep it this way, but I'd recommend to change to either
> have_xtradb.inc or (better) not use innodb at all and test with myisam
> and memory engines.
>
> with have_innodb.inc the test will be run twice - for innodb and for
> xtradb, which is a waste of time, because it doesn't do anything
> engine-specific here.

And with have_xtradb.inc it won't run for those who doesn't build
XtraDB. So I'd suggest to not use that option unless you want to use
some XtraDB-specific features.


Pavel


References