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
F3396817: D8514.id28709.diff
Sun, Dec 1, 2:36 PM
Unknown Object (File)
Fri, Nov 29, 12:58 PM
Unknown Object (File)
Sat, Nov 23, 2:30 PM
Unknown Object (File)
Sat, Nov 23, 12:50 PM
Unknown Object (File)
Sat, Nov 23, 12:50 PM
Unknown Object (File)
Sat, Nov 23, 12:50 PM
Unknown Object (File)
Oct 27 2024, 2:05 PM
Unknown Object (File)
Oct 27 2024, 2:05 PM
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
No Lint Coverage
Unit
No Test Coverage