Page MenuHomePhabricator

Rewrite CommitQuery to use UNION for performance
ClosedPublic

Authored by epriestley on Oct 20 2017, 10:42 PM.
Tags
None
Referenced Files
F15419872: D18722.id44942.diff
Fri, Mar 21, 9:35 AM
F15416478: D18722.diff
Thu, Mar 20, 11:25 AM
F15412099: D18722.id.diff
Wed, Mar 19, 11:18 AM
F15404641: D18722.diff
Tue, Mar 18, 8:31 AM
F15334445: D18722.id44942.diff
Sat, Mar 8, 9:13 AM
Unknown Object (File)
Feb 9 2025, 4:18 AM
Unknown Object (File)
Jan 31 2025, 4:23 AM
Unknown Object (File)
Jan 28 2025, 12:23 AM
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