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)
Sun, Feb 9, 4:18 AM
Unknown Object (File)
Fri, Jan 31, 4:23 AM
Unknown Object (File)
Tue, Jan 28, 12:23 AM
Unknown Object (File)
Jan 17 2025, 8:57 PM
Unknown Object (File)
Jan 17 2025, 3:40 PM
Unknown Object (File)
Jan 15 2025, 11:30 PM
Unknown Object (File)
Jan 1 2025, 10:24 PM
Unknown Object (File)
Dec 30 2024, 7:45 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