This can lead to duplicate settlement entries where a single order was only settled once but the transaction was mined once, then the chain followed a different fork and the transaction was mined again with a different logIndex. Conceptually, I believe our usage is correct but go-ethereum's implementation is wrong.
See ethereum/go-ethereum#2028
Geth's implementation is unlikely to change, so we will need to implement a workaround.
This can lead to duplicate settlement entries where a single order was only settled once but the transaction was mined once, then the chain followed a different fork and the transaction was mined again with a different
logIndex. Conceptually, I believe our usage is correct but go-ethereum's implementation is wrong.See ethereum/go-ethereum#2028
Geth's implementation is unlikely to change, so we will need to implement a workaround.