Page MenuHomePhabricator

Diffusion - maybe fix import spam bug
ClosedPublic

Authored by btrahan on Oct 20 2014, 10:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:08 AM
Unknown Object (File)
Thu, Mar 28, 1:25 AM
Unknown Object (File)
Mon, Mar 25, 7:10 PM
Unknown Object (File)
Mon, Mar 25, 1:48 PM
Unknown Object (File)
Feb 12 2024, 8:53 AM
Unknown Object (File)
Feb 11 2024, 12:28 PM
Unknown Object (File)
Jan 6 2024, 7:00 PM
Unknown Object (File)
Dec 14 2023, 1:34 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.