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
Unknown Object (File)
Thu, Jul 4, 7:12 PM
Unknown Object (File)
Thu, Jul 4, 12:08 PM
Unknown Object (File)
Thu, Jul 4, 12:08 PM
Unknown Object (File)
Thu, Jul 4, 12:08 PM
Unknown Object (File)
Sun, Jun 30, 1:19 PM
Unknown Object (File)
Wed, Jun 26, 4:03 PM
Unknown Object (File)
Wed, Jun 26, 8:04 AM
Unknown Object (File)
Tue, Jun 25, 2:58 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