Page MenuHomePhabricator

Rewrite CommitQuery to use UNION for performance
ClosedPublic

Authored by epriestley on Oct 20 2017, 10:42 PM.
Tags
None
Referenced Files
F18848564: D18722.id44947.diff
Thu, Oct 30, 5:19 AM
F18752068: D18722.diff
Oct 4 2025, 1:58 PM
F18620777: D18722.diff
Sep 15 2025, 6:18 AM
F18462242: D18722.id44942.diff
Sep 1 2025, 9:52 PM
F18461798: D18722.id44947.diff
Sep 1 2025, 9:08 PM
F18459333: D18722.diff
Sep 1 2025, 4:47 PM
F18457873: D18722.id.diff
Sep 1 2025, 1:18 PM
F18457497: D18722.id44947.diff
Sep 1 2025, 12:41 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