[Bug] wallet update states and subscribes to a HUGE number of already spent coins on restart

[iliasorazov]

What happened?

The wallet is very active but only has 2-6 unspent coins now.

The restart takes ~20 minutes and there are following lines in the log:

INFO Successfully subscribed and updated 852 puzzle hashes
INFO new coin state received (1-900/ 1000)
INFO new coin state received (901-1000/ 1000)
<skipped 226 lines>
INFO new coin state received (1-900/ 1000)
INFO new coin state received (901-1000/ 1000)
INFO new coin state received (1-556/ 556)
INFO Successfully subscribed and updated 115556 coin ids

It seams that coins comes from the interested_coins table which has 115556 records now and there is only one place in the code where these coins are removed:

chia-blockchain/chia/wallet/wallet_state_manager.py

Lines 1842 to 1843 in 6c8259c

	if record.coin_type == CoinType.CLAWBACK:
	await self.interested_store.remove_interested_coin_id(coin_state.coin.name())

But what about normal coins? What about rewards? Why the wallet is interested in already spent coins and takes forever to restart? This doesn't seems right.

Any suggestions?

Version

2.1.1

What platform are you using?

Linux

What ui mode are you using?

CLI

Relevant log output

No response

[wjblanke]

We are planning to improve how the lite wallet syncs if it has existing state and has a trusted node.

[iliasorazov]

Does it mean that a wallet can not be sure that the coin reported as spent is actually spent based on the information provided by the node and this is why it resyncs all the coins states every time it restarts? Why it removes CLAWBACK coins from the list of interested coins then?

[iliasorazov]

The last time wallet took more than an hour to start. The increase in time is caused by high network load due to dust storm but the root cause is very very inefficient wallet bootstrap procedure. Is there any simple solution you can suggest? How should I patch the code not to broke it and get rid of these massive spent coins synchronization?

[github-actions]

This issue has not been updated in 14 days and is now flagged as stale. If this issue is still affecting you and in need of further review, please comment on it with an update to keep it from auto closing in 7 days.

[iliasorazov]

Hey! The issue is very actual

[iliasorazov]

@wjblanke can we expect performance improvements in the nearest months? In the next release.

[github-actions]

This issue has not been updated in 14 days and is now flagged as stale. If this issue is still affecting you and in need of further review, please comment on it with an update to keep it from auto closing in 7 days.

[wjblanke]

Running a local node (which is thus trusted) will speed up the wallet syncing process, since coins will not need to be validated. We are working on lite wallet improvements for remote untrusted nodes.

[iliasorazov]

Every transfer a wallet made adds a coin into the interested_coins table. The table is never cleared up by the wallet itself, even when a coin is spent. The list of coins is only growing and all those coins are subscribed on restart causing the huge slow down. This is obviously a bug. Currently we circumvent the issue by clearing up or ignoring the interested_coins in the code.