Improve the performace of LeaseOutput
#8961
Merged
Commits on Aug 9, 2024
-
-
lnrpc+lnwallet: replace
FetchInputInfowith new methodsThis commit replaces the usage of `FetchInputInfo` with `FetchOutpointInfo` and `FetchDerivationInfo` to remove unncessary fetching of the derivation path.
-
-
lntest+lnwallet: remove the method
FetchInputInfoThis method is no longer used. In addition, the `Derivation` field on the `Utxo` is also removed to avoid nil dereference.
-
lnrpc: fetch utxo info in
lockInputsThis commit prepares the following commit where we change the `LeaseOutput` to be more efficient.
-
mulit: remove
ListLeasedOutputsinLeaseOutputThis commit removes the call toe `ListLeasedOutputs` in `LeaseOutput` - the returned info from `ListLeasedOutputs` can easily be accessed via `FetchInputInfo` and this info is only used at one callsite. `ListLeasedOutputs` then becomes unnecessary here, plus it's very slow and needs to be refactored in `btcwallet` instead.
-
itest: remove redundant call to
ht.WaitForBlockchainSyncThis check has already been done when mining blocks.
-
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.