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

Estimating fees on empty Shelley wallet takes 5-6s #2170

Closed
piotr-iohk opened this issue Sep 23, 2020 · 2 comments
Closed

Estimating fees on empty Shelley wallet takes 5-6s #2170

piotr-iohk opened this issue Sep 23, 2020 · 2 comments
Labels
Bug SEVERITY:LOW Small defects which does not prevent any crucial functionality to work.

Comments

@piotr-iohk
Copy link
Contributor

Context

According to benchmarks since nb #636 (revision 76b3a6d) fee estimation time increased from ~1.5s to 5-6s on empty Shelley wallet.

Screenshot from 2020-09-23 12-44-18

Information -
Version 2020.9.11 (git revision: ffeca1d9e0f302111588c7890785051b6cc6b0da)
Platform Linux, all?
Installation from source

Steps to Reproduce

  1. Trigger fee estimation on empty Shelley wallet

Expected behavior

Should return not_enough_money right away.

Actual behavior

Takes 5-6s on nightly benchmarks.
Note for comparison, it takes ~3-4s for 0.4% seq wallet.

(On my local machine I got ~3s for empty wallet, and ~1s on small wallet with funds)


Resolution


QA

@piotr-iohk piotr-iohk added Bug SEVERITY:LOW Small defects which does not prevent any crucial functionality to work. labels Sep 23, 2020
@Anviking
Copy link
Collaborator

Anviking commented Oct 1, 2020

Looking at the commit history around 76b3a6d https://github.com/input-output-hk/cardano-wallet/commits/master?after=647632e17f1a4add822ab76aef01ba6772a7b813+244&branch=master,

it seems #2112 was merged just before, which seems suspicious

@piotr-iohk
Copy link
Contributor Author

piotr-iohk commented Nov 27, 2020

Resolved with #2348.

benchmark results on branch -> https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/743.

E.g.:

1-wallets-to-0.1:
"estimateFeesTime":"8.950 ms"
used to be = 9.975 s

100-wallets-to-0.01:
"estimateFeesTime":"117.2 ms"
used to be = 19.7 s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug SEVERITY:LOW Small defects which does not prevent any crucial functionality to work.
Projects
None yet
Development

No branches or pull requests

2 participants