eth/filters: add timestamp to derived logs

[s1na]

Implements ethereum/execution-apis#639.

[joshstevens19]

This will be huge when it gets merged 🔥

[MariusVanDerWijden]

I don't really like the idea of adding more redundant fields to the logs. The receipt itself already holds blockhash and blocknumber so the logs having them kinda unnecessarily duplicates the data. Now we are adding the timestamp as well. I think the better approach would be to rework our json api s.th. the whole receipt is always returned and you don't need these unnecessary fields in the log anymore.

Not opposed to merging this, just my 2cents around the spec

edit: also Txindex and Txhash

[s1na]

I don't really like the idea of adding more redundant fields to the logs. The receipt itself already holds blockhash and blocknumber so the logs having them kinda unnecessarily duplicates the data. Now we are adding the timestamp as well. I think the better approach would be to rework our json api s.th. the whole receipt is always returned and you don't need these unnecessary fields in the log anymore.

The idea is to exactly avoid the extra roundtrip for fetching the block. While processing the logs we will anyway have to fetch at least the header and might as well return this data when there is clear demand for it.

I think the better approach would be to rework our json api s.th. the whole receipt is always returned and you don't need these unnecessary fields in the log anymore.

I'm not sure I understand the alternative. If you mean return the whole receipt when people query for logs: it still doesn't contain the timestamp.

[bmoyroud]

Thanks for opening this @s1na - having blockTimestamp would be a big win as discussed.

It would avoid the extra call to eth_getBlockByNumber for each log & speed up indexing.