Echo doing master queries on simple GET requests
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	aaron
	Mar 30 2015, 6:22 PM

Description

forceprofile=1 sometimes shows:

query-m: SELECT etp_user,etp_page,etp_event FROM echo_target_page WHERE etp_user = 'X'

This should be able to just use slaves instead.

	Subject	Repo	Branch	Lines +/-
	Defer the "seen time" updates on GET requests	mediawiki/extensions/Echo	master	+3 -1

		Status	Subtype	Assigned	Task
		Resolved		aaron	T88445 MediaWiki active/active datacenter investigation and work (tracking)
		Resolved		aaron	T94448 Echo doing master queries on simple GET requests

aaron raised the priority of this task from to Needs Triage.

aaron updated the task description. (Show Details)

aaron added subscribers: aaron, EBernhardson.

Catrope set Security to None.

Was that the whole query? I can't find where it might be coming from in Echo's current codebase.

EBernhardson triaged this task as Medium priority.Apr 6 2015, 5:55 PM

Change 237530 had a related patch set uploaded (by Aaron Schulz):
Defer the "seen time" updates on GET requests

Change 237530 merged by jenkins-bot:
Defer the "seen time" updates on GET requests

"+channel:DBPerformance +message:*Echo* -message:*CentralAuth* -message:*(SqlBag|markRead)*" shows nothing. So this should be fixed in master.