bitcoinjs / indexd

An external bitcoind index management service module
ISC License
53 stars 23 forks source link

Suitability in bitcoinjs organization #5

Closed dcousens closed 7 years ago

dcousens commented 7 years ago

Thoughts @fanatid? @jprichardson

This is an alternative to insight or the like, in that it uses the mainline https://github.com/bitcoin/bitcoin RPC and software, not a modified version.

It still uses the txindex in bitcoin-core (as moving that out would duplicate a lot of data), and we can either use the soon-to-be-added unspents index as in https://github.com/bitcoin/bitcoin/pull/9806, or we can trivially add our own (probably ideal tbh).

jprichardson commented 7 years ago

Yeah, I think it makes sense to move over.

My biggest question would be: what was the biggest motivator for you writing it?

dcousens commented 7 years ago

@jprichardson I don't like running https://github.com/bitcoin/bitcoin forks; I don't have the manpower to vett my changes continuously, or that of an un-maintained fork.

I wanted something that I could easily play with, add indexes, remove indexes, without dabbling with my consensus mechanism. Easily add leveldb address-clustering indexes, etc.

I wanted something easy to review, less than 600 lines of code (less, ideally).

I wanted something to replace systems quickly becoming legacy in various production environments.

I know this is a common problem in the community, where that ol' addrindex fork is still used, and keeps some companies from upgrading their bitcoin software.

jprichardson commented 7 years ago

Ah, makes sense!

dcousens commented 7 years ago

Moved :+1: