Page MenuHomePhabricator

updateNotificationTimestamp() should batch queries
Closed, ResolvedPublic

Description

If a page is watched my many users, then the UPDATE query should be batched. Currently this runs as a post-send update in idle-transaction mode. It shouldn't be to hard to add batching.

Event Timeline

aaron created this task.May 6 2016, 11:54 PM
Restricted Application added a subscriber: Zppix. · View Herald TranscriptMay 6 2016, 11:54 PM

Change 287297 had a related patch set uploaded (by Aaron Schulz):
Batch updateNotificationTimestamp() UPDATE queries

https://gerrit.wikimedia.org/r/287297

Change 290292 had a related patch set uploaded (by Umherirrender):
Batch updateNotificationTimestamp() UPDATE queries (without wl_id)

https://gerrit.wikimedia.org/r/290292

Change 290292 merged by jenkins-bot:
Batch updateNotificationTimestamp() UPDATE queries (without wl_id)

https://gerrit.wikimedia.org/r/290292

aaron closed this task as Resolved.May 23 2016, 10:14 PM

Change 287297 abandoned by Aaron Schulz:
[DNM] Make updateNotificationTimestamp() use the PK

https://gerrit.wikimedia.org/r/287297

wl_id can now be used safely on wmf.