SUMO sending all queries for 'sessions' table to the master

RESOLVED FIXED in 0.6

Status

support.mozilla.org
Knowledge Base Software
--
critical
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: xb95, Assigned: laura)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: sumo_only)

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
The SUMO site is sending all SELECT queries on the 'sessions' table to the master and not using the slaves.  The slaves feel underutilized.

This is leading to the master paging IT 3-5 times a day because it is bogging down with these queries.  This is a large table, 1.7G, and the DB is not able to efficiently cache it in memory.  Because of that, when enough of these queries come in, the disks hit 100% utilization, and the server performance bogs down for a few minutes.

Would be great to get these queries split out to the slaves if we can.  Thanks!
Component: Webdev → Knowledge Base Software
OS: Mac OS X → All
Product: mozilla.org → Sumo
QA Contact: webdev → kb-software
Hardware: PC → All
Version: other → unspecified
(Assignee)

Comment 1

10 years ago
Going to try to get these into Tuesday's code freeze.
Target Milestone: --- → 0.6

Comment 2

10 years ago
Can I mark this blocking?  Getting wake up pages on this blows.
Target Milestone: 0.6 → ---
Target Milestone: --- → 0.6
Severity: normal → critical
(Assignee)

Comment 3

10 years ago
You can, but it won't go out tonight, 'cos if I make a change like this without
QA to tonight's push Justin's head will explode.

Updated

10 years ago
Assignee: nobody → laura
(Assignee)

Comment 4

10 years ago
Created attachment 327704 [details] [diff] [review]
Force ADODB to use slaves for this query.

I feel there has to be a less hackish way than this, but it works.
Attachment #327704 - Flags: review?(nelson)

Comment 5

10 years ago
Comment on attachment 327704 [details] [diff] [review]
Force ADODB to use slaves for this query.

let's get this staged
Attachment #327704 - Flags: review?(nelson) → review+

Comment 6

10 years ago
this is in r16609 (trunk) and r16610 (production)
(Assignee)

Comment 7

10 years ago
Thanks for committing Nelson!
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Keywords: push-needed
Resolution: --- → FIXED
Keywords: push-needed

Updated

9 years ago
Whiteboard: tiki_feature
I don't think this one should be upstreamed as issue 448497 reverts it. Also, memcache can now be used for sessions.

Updated

9 years ago
Whiteboard: tiki_feature → sumo_triage

Updated

9 years ago
Whiteboard: sumo_triage → sumo_only
You need to log in before you can comment on or make changes to this bug.