If we reach maximum call stack fail transaction instead of debug log

[jak-pan]

We are logging assertion in this case. We should fail the tx instead as you are probably doing something you shouldn't and this could cause issues with tracking of transactions.

[dmoka]

Are we sure that failing the transaction is better than just logging the error?

My concern is that event emitting should not block trades.

If we could receive a timely notification of the error log, we could react promptly. In the worst case, our events may become corrupted.

However, if we fail the transaction, the worst-case scenario is that the trades are blocked.