Depends on D5070
In response to this issue, we should implement the code that will delegate the work away from the reactors to separate threads.
Here, I implement using the worker in the write reactors.
Details
Details
Summary
Test Plan
Services build as usual.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Herald added a reviewer: • jon. · View Herald TranscriptSep 6 2022, 7:22 AM2022-09-06 07:22:33 (UTC-7)
• karol edited the summary of this revision. (Show Details)Sep 6 2022, 7:28 AM2022-09-06 07:28:32 (UTC-7)
• karol added a parent revision: D5070: [services] Rust Integration - c++ - Implement worker in read reactor.
Harbormaster completed remote builds in B11928: Diff 16374.Sep 6 2022, 7:32 AM2022-09-06 07:32:44 (UTC-7)
Harbormaster completed remote builds in B11944: Diff 16395.Sep 7 2022, 2:27 AM2022-09-07 02:27:25 (UTC-7)
This revision now requires changes to proceed.Sep 9 2022, 5:33 AM2022-09-09 05:33:54 (UTC-7)
Harbormaster completed remote builds in B12135: Diff 16657.Sep 14 2022, 3:10 AM2022-09-14 03:10:42 (UTC-7)
Comment Actions
Assuming that this type of capture is correct (see D5070 where this will be clarified)
This revision is now accepted and ready to land.Sep 15 2022, 7:59 AM2022-09-15 07:59:46 (UTC-7)
Harbormaster completed remote builds in B12178: Diff 16706.Sep 15 2022, 9:11 AM2022-09-15 09:11:19 (UTC-7)
Harbormaster completed remote builds in B12241: Diff 16793.Sep 19 2022, 1:08 AM2022-09-19 01:08:35 (UTC-7)
Closed by commit rCOMM82f40e12ac63: [services] Rust Integration - c++ - Implement worker in write reactor (authored by • karol). · Explain WhySep 19 2022, 1:11 AM2022-09-19 01:11:55 (UTC-7)
This revision was automatically updated to reflect the committed changes.