Page MenuHomePhabricator

Rewrite CommitQuery to use UNION for performance
ClosedPublic

Authored by epriestley on Oct 20 2017, 10:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 6:54 PM
Unknown Object (File)
Mar 23 2024, 11:36 AM
Unknown Object (File)
Mar 21 2024, 7:53 AM
Unknown Object (File)
Feb 17 2024, 8:01 PM
Unknown Object (File)
Feb 8 2024, 2:09 PM
Unknown Object (File)
Feb 2 2024, 11:53 AM
Unknown Object (File)
Jan 25 2024, 12:37 AM
Unknown Object (File)
Jan 9 2024, 4:39 PM
Subscribers
None

Details

Summary

Ref T12680. See PHI167. See that task for discussion.

Rewrite DiffusionCommitQuery to work more like DifferentialRevisionQuery, and use a UNION to find "all revisions you need to audit OR respond to".

I tried to get this working a little more cleanly than RevisionQuery does, and can probably simplify that now.

Test Plan

Poked at the UI locally without hitting any apparent issues, but my local data is pretty garbage at this point. I'll take a look at how the query plans work on secure.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable