Consider removing IPLDization of logs #3

Open
opened 2023-10-04 22:15:54 +00:00 by telackey · 0 comments
Member

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.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cerc-io/ipld-eth-db#3
No description provided.