Remove cmd/dropwtxmgr and doco references to it. #434

Merged
merged 1 commit into from Dec 6, 2016

Projects

None yet

2 participants

@jrick
Member
jrick commented Dec 6, 2016

This tool is no longer necessary to force rescans with the new rescan
RPCs available in both the JSON-RPC and gRPC servers, and the new
versions are better anyways (can start the rescan at any block height,
and for the gRPC version, the RPC returns a stream of progress
notifications).

For situations where problematic transation data must be removed from
the wallet database, a reseed should be performed. It is better to
have a single method of fixing these wallets instead of several.

The wtxmgr namespace is now required to exist when opening a wallet.
Older versions of the dropwtxmgr tool should not be used on the
database anymore.

Closes #433.

@jrick jrick Remove cmd/dropwtxmgr and doco references to it.
This tool is no longer necessary to force rescans with the new rescan
RPCs available in both the JSON-RPC and gRPC servers, and the new
versions are better anyways (can start the rescan at any block height,
and for the gRPC version, the RPC returns a stream of progress
notifications).

For situations where problematic transation data must be removed from
the wallet database, a reseed should be performed.  It is better to
have a single method of fixing these wallets instead of several.

The wtxmgr namespace is now required to exist when opening a wallet.
Older versions of the dropwtxmgr tool should not be used on the
database anymore.

Closes #433.
091b500
@jcvernaleo

I don't remember the last time I used that instead of restore from seed anyway.
OK

@jrick jrick merged commit b9ef7b1 into decred:master Dec 6, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jrick jrick deleted the jrick:rmdropwtxmgr branch Dec 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment