add mock executor #300
Closed
add mock executor #300
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* feat: adding list command to check ACL status on acl folder. also function can be reused by other parts of the code. * feat: adding ListComponent calls in cdk-erigon added call to the function for it to log in parts of the code, this will be reviewed. * docs: adding readme updates on acl, making it more clear on how to use. * feat: acl transaction history buildup creating functions to interact with table as binary, and other helper functions that we may need. * feat: build up transaction inserter, second part. * feat: enhance ACL listing with policy transaction logging and refactor logging imports changing log structure, cause it's failing. changed the function at txpool so I can get the string to put in the log at the CLI level since it doesn't log too. * chore: changing call to log it. * feat: update logging for policy transactions and fix a transaction insertion error * feat: add update operation to policy transactions and log updates in UpdatePolicies * feat: insert policy refactor and fix update error. fixing update method which was exiting on panic, also improved the function of insert to be able to insert an array improving code quality * feat: adding log count setup to the mode acl function ( which basically is the config updater now) * chore: changing config value on key to string, for easier reading when debugging// looking for it. not an issue for usage, only a improvement. * docs: update README to include optional log_count parameter for acl mode command * chore: adding logging level part back. * fix: we had an issue in the test when running in parallel it could either remove or add the policy at the same time someone else was checking for another scenario to fix I added the adding flow, to updating/changing tx before making the update with the same asserting checks for correct adding.
…s in this part of the code.
…uencer-log-messages-when-overflowing-counters-are-causing-confusion feat: adding new function that will help us log a better counter stats in this part of the code.
…Hermez/cdk-erigon into add-rpcs-return-finished-block
…nHermez#1345) * tweak(acl): log recent ACL change transactions (default 10) * chore: removing old usage of loc count in ACL cli fixing it --------- Co-authored-by: Arthur Abeilice <[email protected]>
* feat: add forkid13 and activate modexp * feat: add secp256verify precompile * fix: do not mutate original r, s,x,y values
…-return-finished-block feat: change RPCs to use latest finished block
* feat: introduce cli flag, JSON unmarshal logic * Add event handler for AddNewRollupTypeTopicBanana * test: move the L1InjectedBatch UnmarshalJSON unit test and implement MarshalJSON * feat: introduce RollupMetadata struct * chore: reuse the code for AddNewRollupTypeTopic * feat: detect if injected batch is already in the genesis file * feat: rename the flag * chore: simplify * Added debug logs and unmarshal L1InjectedBatch instead * Remove RollupMetadata and lower log level to debug * fix: address comments --------- Co-authored-by: Jerry <[email protected]>
…or-tests-fork13 feat: add test vectors fork13
…xPolygonHermez#1350) * Change balance and nonce in zero's tracer output from decimal to hex * test: add unit test for TxnTrace_MarshalJSON --------- Co-authored-by: Stefan Negovanović <[email protected]>
Signed-off-by: Ji Hwan <[email protected]>
* batches support fork 13 * Add ImpossibleForkId * Add IsForkID13Durian for chain rules (0xPolygonHermez#1362) --------- Co-authored-by: Scott Fairclough <[email protected]> Co-authored-by: zhangkai <[email protected]>
Signed-off-by: Ji Hwan <[email protected]>
Signed-off-by: Ji Hwan <[email protected]>
… jihwan/acl-tests Signed-off-by: Ji Hwan <[email protected]>
feat(witness-cache) write witness cache per verified batch and offset wip feat(witness-cache) truncate witness cache properly feat(witness-cache) ad purge witness cache to prestart feat(web) removed logs for skipping witness cache stage feat(witness-cache) only cache trimmed and only get cached trimmed on rpc call feat(witness-cache) revert buildsmtfromwitness to take single witness
…nHermez#1761) fix(rpc): allow block hash for eth get receipts
…-getblockhash fix(rpc): stop getblockhash from crashing when sending hash (0xPolygonHermez#1761)
…tness-cache-offset tweak(witness-cache): cache offset before and after
…deBigInt() (0xPolygonHermez#1772) (0xPolygonHermez#1780) * improve the performance of HashContractBytecodeBigInt() * update go.sum * update poseidongold to v0.0.3 Co-authored-by: Dumi Loghin <[email protected]>
…rmez#1781) Co-authored-by: Elliot Hallam <[email protected]>
…ctBytecodeBigInt() (0xPolygonHermez#1772)" This reverts commit 607ab13.
…PolygonHermez#1799) * feat: check counters overflow on eth_estimateGas * fix: flag zkevm.disable-virtual-counters=true disables counters for gas estimation
Merge upstream v2.61.17
Co-authored-by: haogeng xie <[email protected]> Co-authored-by: Dumi Loghin <[email protected]>
zjg555543
approved these changes
Mar 11, 2025
giskook
force-pushed
the
kyrin/mock-executor
branch
from
9ea94a3 to
2c08e03
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#297