New search indexing failing because it is updating and inserting to slave db

VERIFIED FIXED in 0.8.1

Status

P1
major
VERIFIED FIXED
10 years ago
9 years ago

People

(Reporter: nkoth, Unassigned)

Tracking

unspecified
0.8.1
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: sumo_only)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

10 years ago
Indexer-forums.php and indexer.php are not writing to the master db in production, so the indexing fails.

Need to refactor code to use master database for writes UPDATE and INSERT

Trevor will setup support-stage to use shadowdb so that these problems can be detected during staging

Comment 1

10 years ago
support-stage is configured to use the staging shadow database now. Previously it was using the master as the shadow, so writes to the shadow worked when they shouldn't.
(Reporter)

Updated

10 years ago
Priority: -- → P1
Summary: New search indexing updating and inserting to slave db → New search indexing failing because it is updating and inserting to slave db
Target Milestone: --- → 0.8.1
(Reporter)

Updated

10 years ago
Blocks: 405028
(Reporter)

Updated

10 years ago
Blocks: 460213
(Reporter)

Comment 2

10 years ago
Created attachment 353874 [details] [diff] [review]
make writes go to master db

this should make writes go to the master
Attachment #353874 - Flags: review?(laura)

Updated

10 years ago
Attachment #353874 - Attachment is patch: true
Attachment #353874 - Attachment mime type: application/octet-stream → text/plain

Updated

10 years ago
Attachment #353874 - Flags: review?(laura) → review-
Comment on attachment 353874 [details] [diff] [review]
make writes go to master db

All your mysql_query() calls need turning into tikilib->query(). You've missed a couple.
(Reporter)

Comment 4

10 years ago
Created attachment 353884 [details] [diff] [review]
better patch with more comprehensive refactoring and forceMaster
Attachment #353874 - Attachment is obsolete: true
Attachment #353884 - Flags: review?(laura)

Updated

10 years ago
Attachment #353884 - Flags: review?(laura) → review+
Committed in trunk r21144, prod branch r21146.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Trevor, is this something you could verify on the backend?
Status: RESOLVED → VERIFIED
Whiteboard: sumo_only
You need to log in before you can comment on or make changes to this bug.