Depends on D5037
In order to be able to handle multiple connections at the same time, we have to store a collection of clients in the rust state instead of having just one.
Here, I'm making the c++ calls to rust for the get client valid.
Details
Details
Summary
Test Plan
cd services yarn run-integration-tests backup
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Herald added a reviewer: • jon. · View Herald TranscriptSep 2 2022, 3:52 AM2022-09-02 03:52:58 (UTC-7)
Harbormaster returned this revision to the author for changes because remote builds failed.Sep 2 2022, 4:01 AM2022-09-02 04:01:57 (UTC-7)
Harbormaster completed remote builds in B11846: Diff 16251.Sep 2 2022, 7:33 AM2022-09-02 07:33:34 (UTC-7)
This revision is now accepted and ready to land.Sep 5 2022, 8:11 AM2022-09-05 08:11:08 (UTC-7)
This revision was landed with ongoing or failed builds.Sep 16 2022, 1:55 AM2022-09-16 01:55:22 (UTC-7)
Closed by commit rCOMM14039f468bb9: [services] Rust Integration - Backup - c++ - Multiple get clients (authored by • karol). · Explain Why
This revision was automatically updated to reflect the committed changes.
Harbormaster completed remote builds in B12204: Diff 16739.Sep 16 2022, 2:06 AM2022-09-16 02:06:33 (UTC-7)