Consider bundling tx effects in getLogsByTag

[nventuro]

In #9623 we saw the need to call getTxEffect while processing logs in order to know the note hashes (new commitments) in a given tx hash, and therefore find the nonce of a note. This results in quite a few roundtrips to the node - it may be better to return the new comittments along with the tx hash in getLogsByTag, since that information is always required in order to perform the task getLogsByTag is designed for.

[Thunkar]

A partial solution to this was implemented in #9794. TxEffects are cached now, since it's sensible to assume we'll have a lot of logs from the same transaction during a sync