Closed Bug 470381 Opened 16 years ago Closed 16 years ago

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

Categories

(support.mozilla.org :: Knowledge Base Software, task, P1)

Product:
support.mozilla.org
Component:
Knowledge Base Software
Type:
task

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: nkoth, Unassigned)

References

Details

(Whiteboard: sumo_only)

Attachments

(1 file, 1 obsolete file)

better patch with more comprehensive refactoring and forceMaster
8.29 KB, patch
laura
: review+
Details | Diff | Splinter Review 
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
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.
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
Blocks: 405028
Blocks: 460213
Attached patch make writes go to master db (obsolete) — — Splinter Review 
this should make writes go to the master
Attachment #353874 - Flags: review?(laura)
Attachment #353874 - Attachment is patch: true
Attachment #353874 - Attachment mime type: application/octet-stream → text/plain
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.

        Attachment #353874 -
        Attachment is obsolete: true

        Attachment #353884 -
        Flags: review?(laura)

    
  

        Attachment #353884 -
        Flags: review?(laura) → review+

    
    

    
  
Committed in trunk r21144, prod branch r21146.
Status: NEW → RESOLVED
Closed: 16 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.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: