Page MenuHomePhabricator

[lib] Introduce a handler that processes the ops
ClosedPublic

Authored by tomek on Mar 21 2024, 8:18 AM.
Tags
None
Referenced Files
F3377978: D11366.id38268.diff
Wed, Nov 27, 8:28 AM
Unknown Object (File)
Tue, Nov 12, 10:39 PM
Unknown Object (File)
Tue, Nov 12, 10:39 PM
Unknown Object (File)
Tue, Nov 12, 10:39 PM
Unknown Object (File)
Tue, Nov 12, 10:39 PM
Unknown Object (File)
Tue, Nov 12, 10:38 PM
Unknown Object (File)
Tue, Nov 12, 9:27 PM
Unknown Object (File)
Oct 18 2024, 7:41 AM
Subscribers

Details

Summary

When an action that doesn't generate ops is dispatched, we can immediately delete it from the queue. In the next diff, we're handling actions that generated some ops.

https://linear.app/comm/issue/ENG-7098/create-a-handler-that-processes-the-ops

Depends on D11284

Test Plan

Modified update window active action to contain an ID and checked if the handler dispatches actions removing the action from the queue.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek requested review of this revision.Mar 21 2024, 8:32 AM
This revision is now accepted and ready to land.Mar 22 2024, 3:30 AM

Use the new approach where we have only one queue and process the ops one by one

tomek edited the summary of this revision. (Show Details)