Optimize eth_getLogs efficiency

[thomas-nguy]

Proposal:

GetLogs performance is quite bad at the moment. It will try to find logs matching the criterias by looping through all the blocks and transactions
It won't work well when the chain grow significantly.

We should use the bloom filter to avoid checking block which are tested negative.

Current behavior: [What currently happens]

Loop through all blocks/transactions/logs

Desired behavior: [What you would like to happen]

Check only blocks which bloom filter returns positive

[leejw51crypto]

which part, can i see the source?

[thomas-nguy]

https://github.com/tharsis/ethermint/blob/main/ethereum/rpc/namespaces/eth/filters/filters.go#L87

It seems bloom filter is used only for single block search (when blockhash is specifiied) but not for range

[leejw51crypto]

i see, check bloom filter for each block?

[fedekunze]

yeah, the whole filter API needs to be refactored, this could be a great start

[fedekunze]

will close this one in favor of #563