Page MenuHomePhabricator

Support an "Ancestors Of: ..." constraint in commit queries
ClosedPublic

Authored by epriestley on May 7 2018, 8:50 PM.
Tags
None
Referenced Files
F19817236: D19431.id46487.diff
Thu, Mar 5, 11:48 AM
F19654655: D19431.id46487.diff
Feb 6 2026, 3:51 PM
F19575610: D19431.id46487.diff
Feb 1 2026, 7:13 AM
F19512617: D19431.id46487.diff
Jan 12 2026, 11:34 PM
F19296507: D19431.id.diff
Dec 23 2025, 12:11 PM
F19259169: D19431.diff
Dec 22 2025, 3:19 PM
F18994120: D19431.id46481.diff
Nov 19 2025, 10:11 AM
F18993315: D19431.diff
Nov 19 2025, 7:45 AM
Subscribers
None

Details

Summary

Ref T13137. See PHI609. An install would like to filter audit requests on a particular branch, e.g. "master".

This is difficult in the general case because we can not apply this constraint efficiently under every conceivable data shape, but we can do a reasonable job in most practical cases.

See T13137#238822 for more detailed discussion on the approach here.

This is a bit rough, but should do the job for now.

Test Plan
  • Filtered commits by various branches, e.g. "master"; "lfs". Saw correct-seeming results.
  • Stubbed out the "just list everything" path to hit the diffusion.internal.ancestors path, saw the same correct-seeming results.

Diff Detail

Repository
rP Phabricator
Branch
history1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20292
Build 27549: Run Core Tests
Build 27548: arc lint + arc unit

Event Timeline

amckinley added inline comments.
src/applications/diffusion/query/DiffusionCommitQuery.php
320

"is has"

This revision is now accepted and ready to land.May 7 2018, 11:24 PM
This revision was automatically updated to reflect the committed changes.