Page MenuHomePhabricator

Synchronize working copies before doing a "bypassCache" commit read
ClosedPublic

Authored by epriestley on Apr 22 2016, 11:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 29 2024, 12:12 AM
Unknown Object (File)
Jan 7 2024, 7:33 AM
Unknown Object (File)
Dec 26 2023, 8:33 PM
Unknown Object (File)
Dec 23 2023, 3:25 PM
Unknown Object (File)
Dec 4 2023, 10:24 PM
Unknown Object (File)
Dec 4 2023, 10:24 PM
Unknown Object (File)
Dec 4 2023, 10:24 PM
Unknown Object (File)
Dec 3 2023, 7:58 AM
Subscribers
None

Details

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.

Diff Detail

Repository
rP Phabricator
Branch
sync1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11895
Build 14942: Run Core Tests
Build 14941: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Synchronize working copies before doing a "bypassCache" commit read.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 22 2016, 2:31 PM
chad requested changes to this revision.Apr 22 2016, 2:31 PM
chad edited edge metadata.
This revision now requires changes to proceed.Apr 22 2016, 2:31 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 22 2016, 2:32 PM
This revision was automatically updated to reflect the committed changes.