tendermint / tm-db

Common database interface for various database backends for Tendermint Core and Cosmos SDK
Apache License 2.0
89 stars 137 forks source link

Tarantool support #192

Open TrashCoder96 opened 3 years ago

TrashCoder96 commented 3 years ago

Hi! Do you plan to support tarantool as db-backend for tendermint? Do you accept PRs with such serious feature from external developers? For our goals we are planning to develop a blockchain with tarantool as database to persist blocks and transactions. Tendermint engine is beautiful for us, but it doesn't supports taranstool. Our team migth try to develop this feature, do you have any restrictions for that?

creachadair commented 3 years ago

Hi @TrashCoder96. We do not have plans to add additional storage backends to tm-db at the moment. Looking forward, we want to consolidate the way Tendermint stores data to better fit the data access patterns the node relies on. RFC 001 Storage Engines and Database Layer outlines what we have in mind, and would be a good place to start if you're interested in this topic.

To your other question: We do accept contributions, and our Contributor Guidelines are a good place to start. In this case, however, I'd suggest starting with a proposal rather than a PR, since adding another storage engine impacts all the existing users of tm-db, which already has a fairly large build footprint. Adding one more could make sense, but it's worth soliciting input before doing a bunch of work.