-
Notifications
You must be signed in to change notification settings - Fork 714
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
[TODO] Wallet startup taking too long (Review). #969
Comments
This one is definitely needed. If you have alot of transactions and use a really old backup it takes forever |
can a debug log be provided to understand the cause of this slow startup?or maybe a way to reproduce this issue |
It's the rescan. Haven't got a wallet you can use to reproduce it. |
For a basic wallet; it looks like about 75% of the startup is grinding through |
Anything been looked into? |
I'll be looking into it this week once I have some free time, looks like the problem is more noticeable when loading a new wallet with the HD addition I have, compared to the normal wallet creation. |
#1012 will improve this. Still lot of work to do in many areas, not one final solution. |
Have started a new project to organize the work (review, refactor, cleanup and improvements) of the whole initialization process. https://github.com/PIVX-Project/PIVX/projects/27 . I will be organizing all of the efforts in the area. If anyone want to be part of it, please create an issue of the problem that want to solve (independently if can solve it or not) and/or the back port that want to do and will add it to the kanban board. Be technical and specific on the description. More hands are more than welcome, meanwhile will continue advancing alone on the area. |
Closing, after all of the past year improvements + 2209 this is not a problem anymore. |
This issue tracker is only for tracking purpose and surely will encompass several commits in different areas of our product.
Basically, the startup is taking too long. Needs a full code review, cleanup and possibly work parallelization.
The text was updated successfully, but these errors were encountered: