Page MenuHomePhabricator
Differential D21449

Make "CommitData" wrap and persist a "CommitRef" record
ClosedPublic
Actions

Authored by epriestley on Aug 12 2020, 8:02 PM.
Tags
None
Referenced Files
F14800721: D21449.id51085.diff
Sat, Jan 25, 8:39 AM
F14800719: D21449.id51048.diff
Sat, Jan 25, 8:39 AM
F14800718: D21449.id.diff
Sat, Jan 25, 8:39 AM
F14800717: D21449.diff
Sat, Jan 25, 8:39 AM
Unknown Object (File)
Tue, Jan 21, 10:26 AM
Unknown Object (File)
Sat, Jan 18, 7:03 AM
Unknown Object (File)
Sat, Jan 18, 2:01 AM
Unknown Object (File)
Sat, Jan 11, 3:16 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch
Commits
rPcebde34425c2: Make "CommitData" wrap and persist a "CommitRef" record
Summary

Ref T13552. Turn "CommitData" into an application-level layer on top of the repository-level "CommitRef" object.

For older commits which will not have a "CommitRef" record on disk, build a synthetic one at runtime. This could eventually be migrated.

Test Plan

Ran "bin/repository reparse --message", browsed Diffusion.

Diff Detail

Repository
rP Phabricator
Branch
cref8
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24863
Build 34300: Run Core Tests
Build 34299: arc lint + arc unit

Event Timeline

epriestley created this revision.Aug 12 2020, 8:02 PM
Harbormaster completed remote builds in B24863: Diff 51048.Aug 12 2020, 8:03 PM
epriestley requested review of this revision.Aug 12 2020, 8:03 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 16 2020, 12:37 AM
Closed by commit rPcebde34425c2: Make "CommitData" wrap and persist a "CommitRef" record (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPcebde34425c2: Make "CommitData" wrap and persist a "CommitRef" record.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
data/
4 lines
repository/
storage/
5 lines
57 lines
worker/
20 lines
commitmessageparser/
46 lines

Diff 51048

View Options

src/applications/diffusion/data/DiffusionCommitRef.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommit.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommitData.php

Loading...
View Options

src/applications/repository/worker/PhabricatorRepositoryCommitParserWorker.php

Loading...
View Options

src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php

Loading...
Phabricator · Built by Phacility