Closed (fixed)
Project:
Signup
Version:
6.x-1.x-dev
Component:
Database
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
1 Dec 2008 at 21:02 UTC
Updated:
16 Dec 2008 at 03:01 UTC
Jump to comment: Most recent file
In working on #336255: CCK datestamp fields totally broken and fields without timezone handling mostly broken I noticed that we're doing a LEFT JOIN for the reminder and autoclose SQL for CCK date* fields. Since we do a separate query for each node type, when we join on the appropriate {content_*} table for the start date, I'm pretty sure we can INNER instead of LEFT JOIN there, which would be better for DB performance. I'll test that after lunch and commit a change if all looks good.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 341331_signup_inner_not_left.1.d5.patch | 2.32 KB | dww |
| #1 | 341331_signup_inner_not_left.1.d6.patch | 963 bytes | dww |
Comments
Comment #1
dwwThese work for me. Anyone else care to look while I go eat? ;)
Comment #2
dwwCommitted to HEAD and DRUPAL-5--2.