GasLimitExceeded on recent transactions

[frol]

https://explorer.testnet.near.org/transactions/3oW9Kjnas1J4MGzFKStpHVj1SSuMMSJ244bDxVNaWLHD

[ailisp]

Why this exceed gas limit? From explorer it attached 300T gas but only used 205T 🤔

[maxzaver]

Why this exceed gas limit? From explorer it attached 300T gas but only used 205T 🤔

300Tgas is the max attached gas. 200Tgas is the limit per single contract execution.

[maxzaver]

At first I thought it is related to super deep reorgs see #329 <#329>, because I haven’t analyzed the contract on the subject of super deep re-orgs. However, Ropsten did no undergo deep reorgs in the past few days. I suspect either of the following happened: * `add_block_header` is not O(1), maybe also because of some unknown bug in `UnorderedMap`. Because this specific client was launched with `finalized_gc_threshold=30k` the issue has surfaced; * Since Ropsten is unstable we could’ve received a malformed header;

…

On Aug 31, 2020, at 3:25 AM, Vlad Frolov ***@***.***> wrote: https://explorer.testnet.near.org/transactions/3oW9Kjnas1J4MGzFKStpHVj1SSuMMSJ244bDxVNaWLHD <https://explorer.testnet.near.org/transactions/3oW9Kjnas1J4MGzFKStpHVj1SSuMMSJ244bDxVNaWLHD> — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#332>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AILKVBZMOS3Y6NFOFSCKPA3SDN3BHANCNFSM4QQLNQSQ>.

[maxzaver]

Assigning to @abacabadabacaba since he is auditing our contracts.

[alexauroradev]

Closed with #479