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, 10:52 AM2022-09-02 10:52:58 (UTC+0)
Harbormaster returned this revision to the author for changes because remote builds failed.Sep 2 2022, 11:01 AM2022-09-02 11:01:57 (UTC+0)
Harbormaster completed remote builds in B11846: Diff 16251.Sep 2 2022, 2:33 PM2022-09-02 14:33:34 (UTC+0)
This revision is now accepted and ready to land.Sep 5 2022, 3:11 PM2022-09-05 15:11:08 (UTC+0)
This revision was landed with ongoing or failed builds.Sep 16 2022, 8:55 AM2022-09-16 08:55:22 (UTC+0)
Closed by commit rCOMM14039f468bb9: [services] Rust Integration - Backup - c++ - Multiple get clients. · Explain Why
This revision was automatically updated to reflect the committed changes.
Harbormaster completed remote builds in B12204: Diff 16739.Sep 16 2022, 9:06 AM2022-09-16 09:06:33 (UTC+0)