celestiaorg / cosmos-sdk

:chains: A Framework for Building High Value Public Blockchains :sparkles:
https://cosmos.network/
Other
19 stars 33 forks source link

Implement domain-specific trace writers for SMT multistore #258

Closed Manav-Aggarwal closed 2 years ago

Manav-Aggarwal commented 2 years ago

In #245 and #248, the trace writer in SMT multistore is modified to accomodate a substore-level traceWriter as well so we can keep track of logs on a substore level. This requires having multiple trace writers in a multistore.

We want to explore reducing these multiple trace writers to a single domain-specific writer that notes down the substore as part of the log operation. This would likely require modifying the traceKV store which is a common store used across all other stores for tracing.

We would like to keep compatibility of traceKV with the other stores while exploring this option.

Manav-Aggarwal commented 2 years ago

We use IAVL tree now. Closed by: https://github.com/celestiaorg/cosmos-sdk-rollmint/pull/1