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, 2:22 PM2022-09-06 14:22:11 (UTC+0)
• karol edited the summary of this revision. (Show Details)Sep 6 2022, 2:28 PM2022-09-06 14:28:25 (UTC+0)
• 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, 2:29 PM2022-09-06 14:29:52 (UTC+0)
Harbormaster completed remote builds in B11943: Diff 16394.Sep 7 2022, 9:26 AM2022-09-07 09:26:59 (UTC+0)
This revision now requires changes to proceed.Sep 9 2022, 12:32 PM2022-09-09 12:32:59 (UTC+0)
Harbormaster completed remote builds in B12134: Diff 16656.Sep 14 2022, 10:10 AM2022-09-14 10:10:33 (UTC+0)
Harbormaster completed remote builds in B12177: Diff 16705.Sep 15 2022, 4:10 PM2022-09-15 16:10:50 (UTC+0)
This revision is now accepted and ready to land.Sep 16 2022, 5:10 PM2022-09-16 17:10:42 (UTC+0)
Harbormaster completed remote builds in B12240: Diff 16792.Sep 19 2022, 8:07 AM2022-09-19 08:07:30 (UTC+0)
Closed by commit rCOMM62f8680c387b: [services] Rust Integration - c++ - Implement worker in read reactor. · Explain WhySep 19 2022, 8:11 AM2022-09-19 08:11:50 (UTC+0)
This revision was automatically updated to reflect the committed changes.