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.

Sign up for GitHub

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

Jump to bottom

Using the witness rework from ZERO PR - modify to not use a new database #190

Open
blondfrogs opened this issue Apr 22, 2022 · 2 comments
Open

Using the witness rework from ZERO PR - modify to not use a new database #190

blondfrogs opened this issue Apr 22, 2022 · 2 comments
Assignees
@blondfrogs
Labels
bug Something isn't working

Comments

@blondfrogs
Copy link
Member

  • This could fix the issue where we are unable to invalidate more than 29 blocks without a segfault occuring.
@blondfrogs blondfrogs added the bug Something isn't working label Apr 22, 2022
@blondfrogs blondfrogs self-assigned this Apr 22, 2022
@TheTrunk
Copy link
Member

29 blocks is most likely tied to our protection of not allowing this long reorganisations. It may be 29 blocks instead of 30.

@blondfrogs
Copy link
Member Author

29 blocks is most likely tied to our protection of not allowing this long reorganisations. It may be 29 blocks instead of 30.

I wish. it is instead an issue with the sapling caching system. and zcash still has this problem.

The PR in WIP does fix this i am pretty sure, but does remove backwards capabilities of wallets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@blondfrogs blondfrogs

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@blondfrogs @TheTrunk and others