Closed
Bug 438696
Opened 17 years ago
Closed 17 years ago
Search Index Insert is flooding the database
Categories
(support.mozilla.org :: Knowledge Base Software, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: oremj, Unassigned)
References
Details
(Whiteboard: sumo_only)
Attachments
(1 file)
|
606 bytes,
patch
|
morgamic
:
review+
laura
:
review+
|
Details | Diff | Splinter Review |
I am seeing the following query happen about 300 times/s in production which is flooding the database.
insert into `tiki_searchindex` (`location`,`page`,`searchword`,`count`,`last_update`) VALUES (?)
We need to take it out before we can put the master/slave patch back in production.
Comment 1•17 years ago
|
||
Attachment #324702 -
Flags: review?(morgamic)
Comment 2•17 years ago
|
||
The tiki_searchindex table contains indexes that won't be used when search is using MySQL fulltext index mode anyway.
Updated•17 years ago
|
Attachment #324702 -
Flags: review?(laura)
Updated•17 years ago
|
Severity: normal → major
Updated•17 years ago
|
Attachment #324702 -
Flags: review?(morgamic) → review+
Comment 3•17 years ago
|
||
Now this makes me wonder: did we not switch to InnoDB? Does InnoDB support MySQL fulltext searches?
Comment 4•17 years ago
|
||
Comment 5•17 years ago
|
||
InnoDB doesn't support FTS, but I left all the tables that had full text indexes as MyISAM for that exact reason.
Comment 6•17 years ago
|
||
Comment on attachment 324702 [details] [diff] [review]
do not index into tiki_searchindex table when MySQL fulltext indexes are being used
Let's stage this. oremj?
Attachment #324702 -
Flags: review?(laura) → review+
Comment 7•17 years ago
|
||
To test this: you can go to log in as an admin user, then goto tiki-admin.php?page=search
and then click on the "Refresh wiki search index now" and "Refresh tracker search index now" links, and see if those nasty queries happen.
| Reporter | ||
Comment 8•17 years ago
|
||
That test case on stage does not cause a huge amount of INSERTs or any INSERTs for that matter.
Comment 9•17 years ago
|
||
Incorporated in https://svn.mozilla.org/projects/sumo/tags/1.0/20080612_r15378.
Updated•17 years ago
|
Keywords: push-needed
Updated•16 years ago
|
Whiteboard: tiki_triage
Comment 11•16 years ago
|
||
Per the discussion in bug 532156, we're keeping our own implementation of Sphinx-based search, which isn't compatible with general tiki needs.
However you choose to deal with search, it's worth making sure that the initial problem here is fixed.
Whiteboard: tiki_discuss → tiki_bug
Updated•16 years ago
|
Whiteboard: tiki_bug → tiki_bug, tiki_upstreamed
Updated•16 years ago
|
Whiteboard: tiki_bug, tiki_upstreamed → tiki_bug
Updated•16 years ago
|
Whiteboard: tiki_bug → sumo_only
You need to log in
before you can comment on or make changes to this bug.
Description
•