Page MenuHomePhabricator

Paginate notifications by event ID, not timestamp
Open, Needs TriagePublic

Description

Right now we paginate notifications by (notification_timestamp, notification_event), but @SBisson suggests we should instead paginate by notification_event. Event IDs should be in timestamp order anyway. That'll make the pagination logic simpler, and should mean we won't need notification_timestamp in the indexes used for pagination.

Event Timeline

Catrope created this task.Aug 26 2016, 1:22 AM
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptAug 26 2016, 1:22 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript