Closed
Bug 618256
Opened 15 years ago
Closed 14 years ago
alter get_collection_list_with_timestamps to avoid mysql crash
Categories
(Cloud Services Graveyard :: Server: Sync, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: tarek, Assigned: tarek)
Details
(Whiteboard: [php: 616435] [qa-])
Attachments
(1 file, 2 obsolete files)
1.88 KB,
patch
|
telliott
:
review+
Atoll
:
review+
|
Details | Diff | Splinter Review |
sync 1.5.101116 includes a collection timestamp query that reliably and
simultaneously crashes the mysql servers in production. it appears to stem
from the subquery i suggested using here. we have monkeypatched in production
a revised query based on the original sync 1.3 query with one single change,
specifically "GROUP BY username, collection". this has proven to be sufficient
to reap some performance reward of using the available indexes, without
crashing the mysql server.
old query still present, and we're having random crashes of mysql in staging when running sync tests, so it's very likely this is the root cause.
Severity: normal → blocker
Assignee | ||
Comment 2•14 years ago
|
||
Attachment #506418 -
Flags: review?(telliott)
Assignee | ||
Updated•14 years ago
|
Attachment #506418 -
Flags: review?(rsoderberg)
Assignee | ||
Comment 3•14 years ago
|
||
Assignee | ||
Comment 4•14 years ago
|
||
added "username" to the group by /
Attachment #506418 -
Attachment is obsolete: true
Attachment #506421 -
Attachment is obsolete: true
Attachment #506422 -
Flags: review?(telliott)
Attachment #506418 -
Flags: review?(telliott)
Attachment #506418 -
Flags: review?(rsoderberg)
Assignee | ||
Updated•14 years ago
|
Attachment #506422 -
Flags: review?(rsoderberg)
Comment on attachment 506422 [details] [diff] [review]
Back to the simpler query
Can you provide the resulting SQL query as sent to MySQL by the select() function? r+ otherwise, pending that
Assignee | ||
Comment 6•14 years ago
|
||
SELECT wbo.collection, max(wbo.modified) AS max_1 FROM wbo WHERE wbo.username = %s AND wbo.ttl > %s GROUP BY wbo.username, wbo.collection
Attachment #506422 -
Flags: review?(rsoderberg) → review+
Updated•14 years ago
|
Attachment #506422 -
Flags: review?(telliott) → review+
Assignee | ||
Comment 7•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•14 years ago
|
Whiteboard: [php: 616435] → [php: 616435] [qa-]
Updated•2 years ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•