This solution can only solve #668 and #669, but cannot solve #667. The final solution needs to be discussed.

It's ok for me to use this as a short term approach. I think we might want to set everything to store instead of keeping "dirty" arrays/maps. This would be a large refactor of the evm module but it also will increase the performance as we won't need to iterate over the dirty entries during EndBlock

It's ok for me to use this as a short term approach. I think we might want to set everything to store instead of keeping "dirty" arrays/maps. This would be a large refactor of the evm module but it also will increase the performance as we won't need to iterate over the dirty entries during EndBlock

Yes, I agree with you, the refactored solution should be able to solve #667 #668 #669

Thanks @summerpro! Could you add a test case to ensure this is not broken in the future?

You're welcome. Ok i will add it tomorrow

@fedekunze Can this pr be merged?