Page MenuHomePhabricator
Differential D15748

Record more details about where a write is taking place while holding a cluster lock
ClosedPublic
Actions

Authored by epriestley on Apr 18 2016, 3:30 PM.
Tags
None
Referenced Files
F18906349: D15748.id.diff
Sat, Nov 8, 5:58 PM
F18889091: D15748.diff
Fri, Nov 7, 8:03 AM
F18821163: D15748.id37948.diff
Wed, Oct 22, 7:54 PM
F18738973: D15748.id37948.diff
Oct 1 2025, 6:43 PM
F18673330: D15748.diff
Sep 25 2025, 10:13 AM
F18638372: D15748.id37950.diff
Sep 17 2025, 10:14 AM
F18164247: D15748.id37950.diff
Aug 15 2025, 6:24 AM
F18153428: D15748.id.diff
Aug 14 2025, 7:36 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T4292: Implement repository replication
Commits
rPf424f9f2d206: Record more details about where a write is taking place while holding a…
Summary

Ref T4292. This will let the UI and future bin/repository tools give administrators more tools to understand problems when reporting or resolving them.

Test Plan
  • Pushed fully clean repository.
  • Pushed previously-pushed repository.
  • Forced write to abort, inspected useful information in the database.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37948.Apr 18 2016, 3:30 PM
epriestley retitled this revision from to Record more details about where a write is taking place while holding a cluster lock.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T4292: Implement repository replication.
chad accepted this revision.Apr 18 2016, 4:28 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 18 2016, 4:28 PM
Closed by commit rPf424f9f2d206: Record more details about where a write is taking place while holding a… (authored by epriestley, committed by epriestley). · Explain WhyApr 18 2016, 6:55 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T10751: Make Phabricator Highly Available.Apr 19 2016, 9:29 PM

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
src/
applications/
diffusion/
ssh/
3 lines
repository/
storage/
10 lines
23 lines

Diff 37950

View Options

resources/sql/autopatches/20160418.repoversion.1.sql

Loading...
View Options

src/applications/diffusion/ssh/DiffusionGitReceivePackSSHWorkflow.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php

Loading...
Phabricator · Built by Phacility