View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0008257 | Taler | wallet-core | public | 2024-01-24 14:16 | 2024-04-15 19:32 |
| Reporter | Florian Dold | Assigned To | Florian Dold | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Target Version | 0.10 | Fixed in Version | 0.10 | ||
| Summary | 0008257: wallet-core checkPeerPushDebit and getWithdrawalDetailsForAmount requests are extremely slow for larger amounts [estimate: 8h] | ||||
| Description | Likely some DB transaction is missing an index. Furthermore we should do caching. And unless the response is *always* very fast, we should consider a way to cancel ongoing requests when the UI doesn't even need the old data anymore. | ||||
| Tags | No tags attached. | ||||
| Likely cause: failure to only look at *active* denomination groups for withdraw (and instead iterating over tons of denomination keys in the DB). | |
| Things still missing for decent performance: * in-mem caching of denom selection * in-mem caching of refresh cost * in-mem caching of the conversion API response * denomination families in the wallet-core DB | |
| commit ca60e97755d91d73b4f5fc3992ddb2db524cd5f2 (HEAD -> master, origin/master, origin/HEAD) Author: Florian Dold <florian@dold.me> Date: Mon Apr 8 22:52:53 2024 +0200 wallet-core: fix cancellation commit 18b77088bfc8699dfff64b70bb65b915f08d0fb6 Author: Florian Dold <florian@dold.me> Date: Mon Apr 8 22:34:42 2024 +0200 -parsing commit 859d5c56c7a5b26e741254d6d1e9c5731a787ae1 Author: Florian Dold <florian@dold.me> Date: Mon Apr 8 22:29:51 2024 +0200 wallet-core: support cancellation for getWithdrawalDetailsForAmount | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-01-24 14:16 | Florian Dold | New Issue | |
| 2024-01-24 14:16 | Florian Dold | Status | new => assigned |
| 2024-01-24 14:16 | Florian Dold | Assigned To | => Florian Dold |
| 2024-01-24 14:19 | Florian Dold | Summary | wallet-core checkPeerPushDebit request is extremely slow for larger amounts => wallet-core checkPeerPushDebit and getWithdrawalDetailsForAmount requests are extremely slow for larger amounts |
| 2024-03-05 22:39 | Christian Grothoff | Note Added: 0021706 | |
| 2024-03-27 20:18 | Florian Dold | Summary | wallet-core checkPeerPushDebit and getWithdrawalDetailsForAmount requests are extremely slow for larger amounts => wallet-core checkPeerPushDebit and getWithdrawalDetailsForAmount requests are extremely slow for larger amounts [estimate: 8h] |
| 2024-03-31 16:14 | Florian Dold | Note Added: 0022065 | |
| 2024-03-31 16:30 | Florian Dold | Note Edited: 0022065 | |
| 2024-04-08 20:55 | Florian Dold | Status | assigned => informing |
| 2024-04-08 20:55 | Florian Dold | Resolution | open => fixed |
| 2024-04-08 20:55 | Florian Dold | Note Added: 0022161 | |
| 2024-04-09 11:11 | Christian Grothoff | Fixed in Version | => 0.10 |
| 2024-04-15 19:32 | Christian Grothoff | Status | informing => closed |