Remove code: No need to execute smart contract before tx admission into mempool

[snail-inO]

🌟 Feature Request

📝 Description

A transaction will be executed twice in the current implementation: 1) before mempool admission, 2) block validation (i.e., when it is to be included in the next block).

I don't find it necessary to execute the smart contract twice, and reducing it to one contract execution should help improve the sequencer's performance.

I suggest removing the code of pre-mempool contract execution.
I did remove the related code and found the zkSync sequencer was still functioning.

🤔 Rationale

Benefits

• Reducing overall computation overhead and improving performance

📋 Additional Context

The code for executing a contract before mempool is in the following link. It is also the code that has been suggested to be removed.

zksync-era/core/node/api_server/src/tx_sender/mod.rs

Line 378 in 761bda1

let execution_output = self

[JitanderVerma]

Jmm