Add a bit of test coverage for bulky vs compact edge data representations
ClosedPublic
Actions

Authored by epriestley on Jan 27 2018, 3:33 PM.

Details

Reviewers

Maniphest Tasks

T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs

Commits

rP1b4017b889e2: (stable) Add a bit of test coverage for bulky vs compact edge data…
rP98402b885b62: Add a bit of test coverage for bulky vs compact edge data representations

Summary

Depends on D18948. Ref T13051. The actual logic ended up so simple that this doesn't really feel terribly valuable, but maybe it'll catch something later on.

Test Plan

Ran test.

Diff Detail

Repository

rP Phabricator

Branch

edge4

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 19232
Build 25980: Run Core Tests
Build 25979: arc lint + arc unit

Event Timeline

epriestley created this revision.Jan 27 2018, 3:33 PM

Harbormaster completed remote builds in B19232: Diff 45448.Jan 27 2018, 3:34 PM

epriestley requested review of this revision.Jan 27 2018, 3:34 PM

amckinley accepted this revision.Jan 29 2018, 7:28 PM

This revision is now accepted and ready to land.Jan 29 2018, 7:28 PM

Closed by commit rP98402b885b62: Add a bit of test coverage for bulky vs compact edge data representations (authored by epriestley). · Explain WhyJan 29 2018, 7:35 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

infrastructure/

edges/

__tests__/

PhabricatorEdgeChangeRecordTestCase.php

158 lines

Diff 45448

View Options

src/__phutil_library_map__.php

View Options

src/infrastructure/edges/tests/PhabricatorEdgeChangeRecordTestCase.php

Add a bit of test coverage for bulky vs compact edge data representationsClosedPublicActions