Differential D20490

Fix the direction of the commit/revision edge
ClosedPublic
Actions

Authored by epriestley on May 1 2019, 3:47 PM.

Tags

None

Referenced Files

	F18765937: D20490.id.diff
	Tue, Oct 7, 1:55 PM

	F18762963: D20490.diff
	Mon, Oct 6, 9:50 PM

	F18716303: D20490.id.diff
	Mon, Sep 29, 11:45 AM

	F18704553: D20490.diff
	Sun, Sep 28, 8:04 AM

	F18657716: D20490.id48873.diff
	Tue, Sep 23, 3:06 AM

	F18652462: D20490.id48873.diff
	Sun, Sep 21, 8:08 AM

	F18570062: D20490.id48882.diff
	Sep 10 2025, 3:07 AM

	F18570059: D20490.id48873.diff
	Sep 10 2025, 3:07 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13276: Fix several audit-adjacent issues, including races when multiple closing commits are discovered at the same time

Summary

Ref T13276. This edge is pointed the wrong way. Point it the right way.

Test Plan

Will verify production works better.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 1 2019, 3:47 PM

Harbormaster completed remote builds in B22746: Diff 48873.May 1 2019, 3:49 PM

epriestley requested review of this revision.May 1 2019, 3:49 PM

amckinley accepted this revision.May 2 2019, 12:50 PM

This revision is now accepted and ready to land.May 2 2019, 12:50 PM

Closed by commit rP5dc4e76eb9f8: Fix the direction of the commit/revision edge (authored by epriestley). · Explain WhyMay 2 2019, 2:28 PM

This revision was automatically updated to reflect the committed changes.

epriestley added an edge: rP5dc4e76eb9f8: Fix the direction of the commit/revision edge.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

worker/

DiffusionUpdateObjectAfterCommitWorker.php

2 lines

Diff 48882

View Options

src/applications/diffusion/worker/DiffusionUpdateObjectAfterCommitWorker.php

Fix the direction of the commit/revision edgeClosedPublicActions