HomePhabricator

Synchronize working copies before doing a "bypassCache" commit read

Description

Synchronize working copies before doing a "bypassCache" commit read

Summary:
Ref T4292. When the daemons make a query for repository information, we need to make sure the working copy on disk is up to date before we serve the response, since we might not have the inforamtion we need to respond otherwise.

We do this automatically for almost all Diffusion methods, but this particular method is a little unusual and does not get this check for free. Add this check.

Test Plan:

  • Made this code throw.
  • Ran bin/repository reparse --message ..., saw the code get hit.
  • Ran bin/repository lookup-user ..., saw this code get hit.
  • Made this code not throw.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T4292

Differential Revision: https://secure.phabricator.com/D15783

Details

Provenance
epriestleyAuthored on Apr 22 2016, 10:58 AM
epriestleyPushed on Apr 22 2016, 3:11 PM
Reviewer
chad
Differential Revision
D15783: Synchronize working copies before doing a "bypassCache" commit read
Parents
rP0f0105e783ef: Send the `aphlict` process log to the `node` log
Branches
Unknown
Tags
Unknown
Tasks
T4292: Implement repository replication
Build Status
Buildable 11901
Build 14952: Run Core Tests