Differential D12398

Fix Conpherence transaction ordering
ClosedPublic
Actions

Authored by epriestley on Apr 13 2015, 7:33 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T7815: Conpherence threads ordering messed up; perf maybe too bad now too?

Commits

Restricted Diffusion Commit
rPd096b9c1517e: Fix Conpherence transaction ordering

Summary

Ref T7815. Pretty sure this is right? Seems OK locally. Also: simpler!

Test Plan

Conpherence is right-side-up.
Transactions on other objects are also still right-side-up.

Diff Detail

Repository

rP Phabricator

Branch

order1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 5293
Build 5311: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 29773.Apr 13 2015, 7:33 PM

epriestley retitled this revision from to Fix Conpherence transaction ordering.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T7815: Conpherence threads ordering messed up; perf maybe too bad now too?.

Herald added a subscriber: epriestley. · View Herald TranscriptApr 13 2015, 7:33 PM

btrahan accepted this revision.Apr 13 2015, 7:34 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Apr 13 2015, 7:34 PM

Closed by commit rPd096b9c1517e: Fix Conpherence transaction ordering (authored by epriestley, committed by epriestley). · Explain WhyApr 13 2015, 7:34 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

query/

ConpherenceTransactionQuery.php

5 lines

Diff 29773

View Options

src/applications/conpherence/query/ConpherenceTransactionQuery.php

Fix Conpherence transaction orderingClosedPublicActions