Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

rename pevm option #200

Merged
merged 4 commits into from
Oct 18, 2024
Merged

rename pevm option #200

merged 4 commits into from
Oct 18, 2024

Conversation

andyzhang2023
Copy link
Contributor

do some args optimizations:

  1. '--parallel2' is remove.
  2. '--parallel' will be used instead of '--parallel2'.
  3. '--parallel-legacy' is added for the old version of pevm.
  4. add log info when enabling parallel evm.

sunny2022da
sunny2022da approved these changes Oct 17, 2024
View reviewed changes
Copy link
Collaborator

@sunny2022da sunny2022da left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

welkin22
welkin22 approved these changes Oct 18, 2024
View reviewed changes
Copy link
Contributor

@welkin22 welkin22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sunny2022da sunny2022da merged commit 2b01ec3 into bnb-chain:feature/TxDAG-PEVM Oct 18, 2024
1 check failed
welkin22 pushed a commit that referenced this pull request Oct 22, 2024
@andyzhang2023 @welkin22
rename pevm option (#200)
f717e27 
Co-authored-by: andyzhang2023 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@welkin22 welkin22 welkin22 approved these changes

@sunny2022da sunny2022da sunny2022da approved these changes

@redhdx redhdx Awaiting requested review from redhdx

Assignees
No one assigned
Labels
pevm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@andyzhang2023 @sunny2022da @welkin22