Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(wallet): use RPC pool connections for non-recovery utxo scanning #4598

Conversation

sdbondi
Copy link
Member

@sdbondi sdbondi commented Sep 1, 2022

Description

  • use RPC pool in wallet connectivity when scanning for UTXOs from the base node peer
  • extra: make potentially long-running ping-peer async

Motivation and Context

Fixes the wallet exceeding the max RPC sessions per peer limit of 10.

09:53 WARN  Rejecting handshake because no more RPC sessions available
09:53 ERROR error=Maximum number of client RPC sessions reached for node ee4baee242d0baffcab6ef5f20

How Has This Been Tested?

Manual, 6 wallets connected to one base node, UTXO scanning and recovery and a number of stress tests - max rpc sessions were used but not exceeded

@stringhandler stringhandler merged commit 7c9e22c into tari-project:development Sep 2, 2022
@sdbondi sdbondi deleted the wallet-utxo-scanning-rpc-pool branch September 2, 2022 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants