Page MenuHomePhabricator
Differential D18947

Write edge transactions in a more compact way
ClosedPublic
Actions

Authored by epriestley on Jan 27 2018, 1:56 PM.
Tags
None
Referenced Files
F19051070: D18947.id45446.diff
Thu, Nov 27, 11:06 PM
F19051067: D18947.id45463.diff
Thu, Nov 27, 11:06 PM
F19042011: D18947.id.diff
Wed, Nov 26, 5:32 PM
F19034876: D18947.diff
Tue, Nov 25, 3:20 PM
F19001558: D18947.id45446.diff
Fri, Nov 21, 3:38 AM
F18855125: D18947.id45444.diff
Nov 1 2025, 3:24 AM
F18851821: D18947.id45465.diff
Oct 31 2025, 4:30 AM
F18844826: D18947.diff
Oct 29 2025, 7:47 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs
Commits
rPdb8ba29925be: (stable) Write edge transactions in a more compact way
rPe5639a8ed90f: Write edge transactions in a more compact way
Summary

Depends on D18946. Ref T13051. Begins writing edge transactions as just a list of changed PHIDs.

Test Plan

Added, edited, and removed projects. Reviewed transaction record and database. Saw no user-facing changes but a far more compact database representation.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 27 2018, 1:56 PM
Harbormaster failed remote builds in B19228: Diff 45444!Jan 27 2018, 1:57 PM
epriestley added a child revision: D18948: Add `bin/garbage compact-edges` to compact edges into the new format.Jan 27 2018, 2:19 PM
epriestley updated this revision to Diff 45446.Jan 27 2018, 2:44 PM
  • Remove literal debugging code.
Harbormaster completed remote builds in B19230: Diff 45446.Jan 27 2018, 2:46 PM
epriestley requested review of this revision.Jan 27 2018, 2:46 PM
amckinley accepted this revision.Jan 29 2018, 7:24 PM
amckinley added inline comments.
src/infrastructure/edges/util/PhabricatorEdgeChangeRecord.php
80

"this is"

86

Oh, I thought we still needed to keep this as a list inside of a map?

This revision is now accepted and ready to land.Jan 29 2018, 7:24 PM
epriestley added a comment.Jan 29 2018, 7:25 PM

Oh, I thought we still needed to keep this as a list inside of a map?

The change ended up a lot simpler than I thought at first, so I just went with a plain list.

epriestley updated this revision to Diff 45463.Jan 29 2018, 7:32 PM
  • Fix missing word in comment.
Harbormaster completed remote builds in B19247: Diff 45463.Jan 29 2018, 7:33 PM
Closed by commit rPe5639a8ed90f: Write edge transactions in a more compact way (authored by epriestley). · Explain WhyJan 29 2018, 7:34 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
transactions/
editor/
26 lines
infrastructure/
edges/
util/
27 lines

Diff 45465

View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
View Options

src/infrastructure/edges/util/PhabricatorEdgeChangeRecord.php

Loading...
Phabricator · Built by Phacility