Skip to content

merge AddressIndex into master #40

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

Open
cryptozeny opened this issue Dec 24, 2019 · 0 comments
Open

merge AddressIndex into master #40

cryptozeny opened this issue Dec 24, 2019 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@cryptozeny
Copy link
Member

cryptozeny commented Dec 24, 2019

addressindex is originally suggested by
https://github.com/bitpay/insight

BTC declined merge into master, but they want to keep another repo.
bitcoin/bitcoin#10370

but its enabled only with -addressindex like txindex=1 and its bool so RAM usage is not affected
https://github.com/sugarchain-project/sugarchain/blob/API-addressindex/src/validation.cpp#L218

ZEC and RES have also addressindex
https://github.com/ResistancePlatform/resistance-core/blob/master/src/addressindex.h

if we merge this AddressIndex into master, i wanna do after Yespower PoW cache is finished. this is safer IMO.
@volbil

@cryptozeny cryptozeny added the enhancement New feature or request label Dec 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants