Consider removing IPLDization of logs #3

New Issue

2023-10-04T22:15:54Z

telackey commented

2023-10-04 22:15:54 +00:00

https://github.com/cerc-io/ipld-eth-db/issues/144

Internalize the logs data field into eth.log_cids. Rename eth.log_cids to eth.logs. Drop the cid column. Will remove some data redundancy that currently exists between eth.log_cids and ipld.blocks. As far I as I know, there is no need to reference or lookup individual logs by CID or expose them over IPFS in this capacity. Individual logs are never hashed into any of the canonical data structures, they exist within a list within a receipt (which is hashed into the receipt trie). This will involve making corresponding changes across the indexing software and ipld-eth-server, so is probably very low priority as the payoff for the amount of work required is low.

https://github.com/cerc-io/ipld-eth-db/issues/144 Internalize the logs data field into `eth.log_cids`. Rename `eth.log_cids` to `eth.logs`. Drop the `cid` column. Will remove some data redundancy that currently exists between `eth.log_cids` and `ipld.blocks`. As far I as I know, there is no need to reference or lookup individual logs by CID or expose them over IPFS in this capacity. Individual logs are never hashed into any of the canonical data structures, they exist within a list within a receipt (which is hashed into the receipt trie). This will involve making corresponding changes across the indexing software and ipld-eth-server, so is probably very low priority as the payoff for the amount of work required is low.

telackey added the Copied from Github label 2023-10-04 22:21:27 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cerc-io/ipld-eth-db#3