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

Fee estimation on shelley wallet when tx amt > available balance takes much longer time #1905

Closed
piotr-iohk opened this issue Jul 14, 2020 · 1 comment
Labels
Bug PRIORITY:HIGH Require immediate attention. SEVERITY:MEDIUM Visible impact on a core functionality or some important performance degradation.

Comments

@piotr-iohk
Copy link
Contributor

piotr-iohk commented Jul 14, 2020

Context

Observed by @alan-mcnicholas on Daedalus Windows on shelley testnet. I am also able to observe on Linux, however the difference in time is not so siginficant.

While fee estimation of normal transaction takes 2-4 seconds, when we try to estimate tx that amt > utxo the estimation takes even 13s after it reaches not_enough_money.

Information -
Version Daedalus / 2020.7.6 (git revision: f1a0511f58f70e33c143aa567f36b549d2aeb01a)
Platform Windows / Linux
Installation Daedalus / from source

Steps to Reproduce

  1. Run fee estimation endpoint with amount > wallet's utxo.

Expected behavior

time should be more or less the same as when estimating regular transaction.

Actual behavior

Fee estimation time is twice, three times as long.


Resolution


QA

@KtorZ KtorZ added Bug SEVERITY:MEDIUM Visible impact on a core functionality or some important performance degradation. PRIORITY:HIGH Require immediate attention. and removed BUG:MINOR labels Aug 12, 2020
@piotr-iohk
Copy link
Contributor Author

No longer observed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug PRIORITY:HIGH Require immediate attention. SEVERITY:MEDIUM Visible impact on a core functionality or some important performance degradation.
Projects
None yet
Development

No branches or pull requests

2 participants