Depends on D5069
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 read 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:11 (UTC-7)
• karol edited the summary of this revision. (Show Details)Sep 6 2022, 7:28 AM2022-09-06 07:28:25 (UTC-7)
• karol added a parent revision: D5069: [services] Rust Integration - c++ - Implement worker in bidi reactor.
• karol added a child revision: D5071: [services] Rust Integration - c++ - Implement worker in write reactor.
Harbormaster completed remote builds in B11927: Diff 16373.Sep 6 2022, 7:29 AM2022-09-06 07:29:52 (UTC-7)
Harbormaster completed remote builds in B11943: Diff 16394.Sep 7 2022, 2:26 AM2022-09-07 02:26:59 (UTC-7)
This revision now requires changes to proceed.Sep 9 2022, 5:32 AM2022-09-09 05:32:59 (UTC-7)
Harbormaster completed remote builds in B12134: Diff 16656.Sep 14 2022, 3:10 AM2022-09-14 03:10:33 (UTC-7)
Harbormaster completed remote builds in B12177: Diff 16705.Sep 15 2022, 9:10 AM2022-09-15 09:10:50 (UTC-7)
This revision is now accepted and ready to land.Sep 16 2022, 10:10 AM2022-09-16 10:10:42 (UTC-7)
Harbormaster completed remote builds in B12240: Diff 16792.Sep 19 2022, 1:07 AM2022-09-19 01:07:30 (UTC-7)
Closed by commit rCOMM62f8680c387b: [services] Rust Integration - c++ - Implement worker in read reactor (authored by • karol). · Explain WhySep 19 2022, 1:11 AM2022-09-19 01:11:50 (UTC-7)
This revision was automatically updated to reflect the committed changes.