Posted by Crell on April 15, 2010 at 7:23pm
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | database system |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
The idea behind "DELAY" is to allow queries that don't need to commit immediately to just kinda happen when they get there. However, it is not part of standard SQL. It doesn't work on PostgreSQL or SQLite. It only works on MySQL. However, it only works on the MyISAM table type. Drupal 7 on MySQL now defaults to InnoDB, where DELAY is ignored. So really, it's an edge case that we don't even use.
Should we just get rid of it and save ourselves the hassle?
Comments
#1
I couldn't find anywhere that we were using delay() in the first place.
#2
Totally. DELAYED means "the day when my database melts down has been delayed by an undefined amount of time". Using INSERT DELAYED adds a lot of interesting problems and altogether you might end up with a lower INSERT performance. More importantly you give up control on what happens with your database. Bad idea.
#3
Yay for less code! :) Committed to HEAD.
#4
Automatically closed -- issue fixed for 2 weeks with no activity.
#5
There is more code to be removed.
I also removed the docs: http://drupal.org/node/310079/revisions/view/773668/1024872
#6
Looks good
#7
Dagnabbit, I thought I got some of those. Thanks, jbrown.
#8
Committed to CVS HEAD. Thanks.
#9
Automatically closed -- issue fixed for 2 weeks with no activity.