Page MenuHomePhabricator

Remove direct calls to LowLevelCommitQuery
ClosedPublic

Authored by epriestley on Jan 6 2015, 5:25 PM.
Tags
None
Referenced Files
F13166837: D11253.diff
Tue, May 7, 6:36 AM
Unknown Object (File)
Fri, May 3, 4:09 AM
Unknown Object (File)
Mon, Apr 29, 8:53 PM
Unknown Object (File)
Mon, Apr 29, 4:25 PM
Unknown Object (File)
Fri, Apr 26, 4:36 PM
Unknown Object (File)
Fri, Apr 26, 4:35 PM
Unknown Object (File)
Fri, Apr 26, 4:35 PM
Unknown Object (File)
Fri, Apr 26, 4:35 PM
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
Branch
lowlevel2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3636
Build 3645: [Placeholder Plan] Wait for 30 Seconds

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.