Add function to update current set of nodes to connect #30

Open
opened 2023-03-08 23:37:49 +00:00 by snegurochka · 2 comments
Member

Original issue: https://github.com/nspcc-dev/neofs-sdk-go/issues/338

We can provide a method to update current set of endpoints. Remaining endpoints will use the same connection, new endpoints produce client.dial() and old endpoints trigger client.close() in the clients. Make sure that active requests should be finished.

Alternative solution
Do nothing and call pool.Close() and pool.New() on application side. We also should check that pool.Close() will not affect active requests.

Original issue: https://github.com/nspcc-dev/neofs-sdk-go/issues/338 We can provide a method to update current set of endpoints. Remaining endpoints will use the same connection, new endpoints produce `client.dial()` and old endpoints trigger `client.close()` in the clients. Make sure that active requests should be finished. **Alternative solution** Do nothing and call `pool.Close()` and `pool.New()` on application side. We also should check that `pool.Close()` will not affect active requests.
fyrchik added the
pool
label 2023-05-02 07:56:25 +00:00
alexvanin self-assigned this 2023-06-01 06:53:52 +00:00
alexvanin changed title from Reload pool configuration on external signal to Add function to update current set of nodes to connect 2023-09-20 13:05:29 +00:00
Owner

We can start with a short prototype to see whether proposed solution is okay. Also if it is possible to wait for all active requests during pool.Close().

We can start with a short prototype to see whether proposed solution is okay. Also if it is possible to wait for all active requests during `pool.Close()`.
alexvanin was unassigned by fyrchik 2024-01-16 11:54:11 +00:00
elebedeva was assigned by fyrchik 2024-01-16 11:54:11 +00:00
achuprov was assigned by fyrchik 2024-01-16 11:56:46 +00:00
elebedeva was unassigned by fyrchik 2024-01-16 11:56:50 +00:00
Owner
One could use https://git.frostfs.info/TrueCloudLab/frostfs-node/src/branch/master/pkg/network/cache/multi.go#L99 for the inspiration
Sign in to join this conversation.
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-sdk-go#30
No description provided.