remove non beneficial check

0xPolygonHermez / zkevm-node

Go implementation of a node that operates the Polygon zkEVM Network

Other

531 stars 685 forks source link

remove non beneficial check #3575

Closed ARR552 closed 5 months ago

ARR552 commented 5 months ago

Closes #3574

What does this PR do?

This PR removes a check that: 1- This check does not give any advantage because L1 information has to prevail over Trusted state information. No matter what 2- This check calls the trusted state while processing the information. This adds a huge delay during the execution and reduces the performace of the synchronizer a lot. 3- Mixes trusted state and L1 synchronization

Reviewers

@joanestebanr
@tclemos

ARR552 commented 5 months ago

After review this PR with @eduadiez we agree to keep the check paying a reduced price. The modifications have been included in PR https://github.com/0xPolygonHermez/zkevm-node/pull/3576