[server] Always filter message queries by message creation time
ClosedPublic
Actions

Authored by ashoat on Mar 6 2022, 3:44 PM.

Details

Reviewers

tomek
atul

Commits

rCOMM97cea65d1b74: [server] Always filter message queries by message creation time

Summary

This solves ENG-812: Improve perf of website load. By limiting the amount of messages that MySQL has to look at in order to resolve fetchMessageInfos, we speed up website load significantly.

On my local environment using the prod database, the query resolves in 0.5s instead of 20s.

Depends On D3348

Test Plan

I downloaded the prod database to my local machine and tested how long the query to load the website took. This change reduced it from 20s to 0.5s

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Mar 6 2022, 3:44 PM

Herald added subscribers: • karol, • adrian. · View Herald TranscriptMar 6 2022, 3:44 PM

ashoat mentioned this in D3350: [lib] Don't assume freshMessageStore always has messages for a thread.Mar 6 2022, 3:45 PM

ashoat added a child revision: D3350: [lib] Don't assume freshMessageStore always has messages for a thread.Mar 6 2022, 3:48 PM

ashoat edited the summary of this revision. (Show Details)

ashoat added a parent revision: D3348: [server] Move fetchMessageInfosSince's currentAsOf param to MessageSelectionCriteria.

Harbormaster completed remote builds in B7168: Diff 10094.Mar 6 2022, 3:55 PM

ashoat requested review of this revision.Mar 6 2022, 3:55 PM

ashoat mentioned this in D3354: [server] Gate global time filter for message queries on codeVersion.Mar 6 2022, 10:16 PM

I realized I should probably gate this behind a hasMinCodeVersion given native needs D3350, D3352, and D3353 to work properly with this change. Just put up D3354 to gate the changes

• benschac added a subscriber: • benschac.Mar 7 2022, 6:52 AM

• benschac added inline comments.

lib/types/message-types.js
471 ↗	(On Diff #10094)	q: Could you add a comment here with the normalized amount of time here? I'm assuming it's two weeks at a glance but wasn't 100%. Also, might just be me but it feels odd that there's a time value in `message-types`