Page MenuHomePhabricator

Make race condition window for Conpherence smaller and rarer
ClosedPublic

Authored by epriestley on Jan 8 2015, 8:38 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 12, 4:05 PM
Unknown Object (File)
Tue, Mar 12, 3:55 PM
Unknown Object (File)
Fri, Mar 8, 11:08 AM
Unknown Object (File)
Fri, Mar 8, 10:49 AM
Unknown Object (File)
Thu, Mar 7, 12:02 PM
Unknown Object (File)
Sun, Mar 3, 6:48 AM
Unknown Object (File)
Feb 1 2024, 6:23 PM
Unknown Object (File)
Jan 28 2024, 6:49 PM
Subscribers

Details

Summary

Ref T6713. This isn't very clean, and primarily unblocks D11143.

After D11143, I have a reliable local race where I submit, get a notification immediately, then get a double update (form submission + notification-triggered update).

Instead, make the notification updates wait for form submissions.

This doesn't resolve the race completely. The notification updates don't block chat submission (only the other way around), so if you're really fast you can submit at the same time someone else sends chat and race. But this fixes the most glaring issue.

The overall structure here is still pretty shaky but I tried to improve things a little, at least.

Test Plan

Chatted with myself, saw 0 races instead of 100% races.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make race condition window for Conpherence smaller and rarer.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, joshuaspence.
joshuaspence edited edge metadata.

Sure, why not

This revision is now accepted and ready to land.Jan 8 2015, 9:07 AM
This revision was automatically updated to reflect the committed changes.