Needs work
Project:
SimpleTest
Version:
8.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
19 Sep 2010 at 01:22 UTC
Updated:
3 Feb 2022 at 11:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
moshe weitzman commentedGood idea, but it is a bit in your face for sites that don't use mysql
Comment #2
moshe weitzman commentedIn other words, UI should only show for mysql sites.
Comment #3
klausisetting status according to #2
Comment #4
sunHm. I tested this only marginally, but based on limited testing on my local Windows box, I didn't see a substantial difference in test execution time when enforcing MyISAM. Perhaps it's only my box, which is actually very slow in many other ways. But before continuing with this patch, we should make sure that it actually makes a difference for someone.
Comment #5
sunIn other words - since this does not lead to a performance increase for me, I'm not going to continue working on this patch, but which of course doesn't mean it couldn't be useful for others. I'll focus on #916570: Allow to run only certain test methods instead (and in general, making more tests use the testing profile).
Comment #6
sunComment #7
sunerrr, why MyISAM in the first place? Tests should run using the MEMORY table engine!
Same patch, different engine.
Comment #8
sunerr, scratch that, chx rightfully clarified on IRC that the MEMORY table engine doesn't support BLOB/TEXT columns. Dang.
Comment #9
sunComment #10
craigjones commentedFollowing. In my experience, when I switched testing over to MyISAM it made a 3 fold difference in the time that tests took to complete, so being able to change it trivially is definitely advantageous.
Comment #11
xen commentedI've just tested this, my findings:
The patch doesn't work on 7.15. It applies fine, and looks fine, but looking at the simpletest* tables while a test is running, they're still InnoDB...
As for the utility of the patch, when fixed: I am looking at this because I am trying to run two tests from services.module, which takes 45-55 minutes per default. Hardcoding MyISAM into includes/database/mysql/schema.inc, this is reduced to 10 minutes. A rather major improvement.
Comment #12
pdrake commentedI fixed the logic bug that prevented this patch from functioning. I also added the ability to choose the MEMORY table engine because various MySQL distributions (MariaDB 5.?, Percona 5.5) now support BLOB/CLOB columns. There are a few restrictions on column ordering and indexes, which are handled within the MySQL driver. This patch is for D7, will need a re-roll for D8.
Comment #14
pdrake commented#12: D7-drupal-support_simpletest_table_engine-915922-12.patch queued for re-testing.
Comment #16
pdrake commentedOh, right, this issue is filed against D8 (obviously, this patch won't apply to D8).
Comment #27
quietone commentedThe simpletest module is obsolete in core. Moving to SimpleTest project.