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.

Tags

None

Referenced Files

	F18818489: D18949.id.diff
	Wed, Oct 22, 12:36 AM

	F18804644: D18949.id45467.diff
	Sat, Oct 18, 10:21 AM

	F18772330: D18949.id45448.diff
	Thu, Oct 9, 1:51 AM

	F18585426: D18949.id45448.diff
	Sep 11 2025, 3:31 PM

	F18580422: D18949.id45467.diff
	Sep 11 2025, 3:14 AM

	F18578372: D18949.diff
	Sep 10 2025, 9:50 PM

	F18509257: D18949.id.diff
	Sep 5 2025, 3:19 AM

	F18502021: D18949.diff
	Sep 4 2025, 10:08 PM

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

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 45467

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