Page MenuHomePhabricator

Remove direct calls to LowLevelCommitQuery
ClosedPublic

Authored by epriestley on Jan 6 2015, 5:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jun 29 2026, 4:23 AM
Unknown Object (File)
Jun 23 2026, 4:17 PM
Unknown Object (File)
Jun 22 2026, 5:27 PM
Unknown Object (File)
Jun 5 2026, 1:36 AM
Unknown Object (File)
May 31 2026, 4:01 AM
Unknown Object (File)
Mar 13 2026, 1:40 PM
Unknown Object (File)
Mar 10 2026, 7:24 AM
Unknown Object (File)
Oct 25 2025, 6:38 AM
Subscribers

Details

Summary

Ref T2783. This cleans up some more of the direct VCS access calls. If the repository is local, this boils down to an in-process call. If not, it uses Conduit to make an intracluster request.

Test Plan

Used reparse.php --message <commit> --trace to observe cluster request.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove direct calls to LowLevelCommitQuery.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

Looks like the new parseCommitWithRef is never called?

Oh, man, yuck. I code review.

epriestley edited edge metadata.
  • Call subclasses correctly.
  • Ran real daemons on a real repo, saw them complete all parsing stages, for real.

This class tree is still a bit messy but I didn't want to go crazy with refactoring it.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Jan 6 2015, 6:01 PM
This revision was automatically updated to reflect the committed changes.