Page MenuHomePhabricator

Diffusion - maybe fix import spam bug
ClosedPublic

Authored by btrahan on Oct 20 2014, 10:31 PM.
Tags
None
Referenced Files
F14065133: D10729.diff
Tue, Nov 19, 3:15 AM
F13987966: D10729.id.diff
Mon, Oct 21, 12:21 PM
F13956454: D10729.id25756.diff
Oct 14 2024, 5:43 AM
Unknown Object (File)
Aug 29 2024, 7:23 PM
Unknown Object (File)
Aug 28 2024, 7:37 PM
Unknown Object (File)
Aug 8 2024, 8:20 PM
Unknown Object (File)
Aug 8 2024, 7:14 PM
Unknown Object (File)
Aug 8 2024, 7:14 PM
Subscribers

Details

Summary

Ref T6350. I just set up a repository to import secure.phabricator.com/P and saw an error in my error logs about this, creating a fatal around when we publish feed stories. this is late enough in the editor code path I could see it firing again and again and again...

Test Plan

the first phabricator commit ever only had one duplicated transaction before i applied this patch. its yet to do it again. otherwise, ask some users with the issue to deploy it and see if it fixes things for them too.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Diffusion - maybe fix import spam bug.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added reviewers: epriestley, chad.
chad edited edge metadata.

Do it live

This revision is now accepted and ready to land.Oct 20 2014, 10:33 PM
This revision was automatically updated to reflect the committed changes.