Page MenuHomePhabricator

Echo doing master queries on simple GET requests
Closed, ResolvedPublic


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.

Event Timeline

aaron raised the priority of this task from to Needs Triage.
aaron updated the task description. (Show Details)
aaron added subscribers: aaron, EBernhardson.

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

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

aaron claimed this task.

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