Page MenuHomePhabricator

[keyserver] Filter fetchThreadInfos queries to only look at viewer's threads
ClosedPublic

Authored by ashoat on Jul 16 2023, 4:30 PM.
Tags
None
Referenced Files
F2908541: D8514.diff
Sun, Oct 6, 2:48 PM
Unknown Object (File)
Fri, Oct 4, 9:50 AM
Unknown Object (File)
Fri, Oct 4, 9:41 AM
Unknown Object (File)
Fri, Sep 27, 5:49 PM
Unknown Object (File)
Fri, Sep 27, 12:08 AM
Unknown Object (File)
Wed, Sep 25, 9:45 PM
Unknown Object (File)
Mon, Sep 23, 9:13 AM
Unknown Object (File)
Mon, Sep 23, 3:59 AM
Subscribers
None

Details

Summary

This attempts strategy 3 outlined in ENG-3877.

Depends on D8513

Test Plan

I started this task by testing various queries in the production environment to check performance.

After implementing the change, I used this patch to check performance on the JS side in my dev environment, and to output raw queries so I could test their performance in production: https://gist.github.com/Ashoat/220bb35da34ac7820867ac874d87beab

I found that for all users other than me, this change significantly sped up fetchThreadInfos.

I then deployed this to production to test performance more reliably. Hopefully so far no issues!

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable