Skip to content

Make batch size as a option#7540

Merged
DaniPopes merged 6 commits into
foundry-rs:masterfrom
ConsumerLayer:batchsize_option
Apr 6, 2024
Merged

Make batch size as a option#7540
DaniPopes merged 6 commits into
foundry-rs:masterfrom
ConsumerLayer:batchsize_option

Conversation

@hoaleee

@hoaleee hoaleee commented Apr 2, 2024

Copy link
Copy Markdown
Contributor

Make batch size as a option.

Motivation

We are developing a network based on zkevm - a new evm platform. We are trying to increase compatibility of MUD Framework and zkevm. We found that with the default batch size of 100, zkevm may not be able to handle it, with the --slow option the waiting time is too long.

Solution

So we added batch size as an option to increase customization compatibility with many chains.

@DaniPopes DaniPopes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

cc @klkvr

Also needs cargo +nightly fmt

Comment thread crates/script/src/broadcast.rs Outdated
Comment thread crates/script/src/lib.rs Outdated
Comment thread crates/script/src/lib.rs Outdated
Comment thread crates/script/src/lib.rs Outdated
Comment thread crates/script/src/broadcast.rs Outdated
hoaleee and others added 5 commits April 5, 2024 17:54
Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
@hoaleee hoaleee requested a review from DaniPopes April 6, 2024 14:17

@DaniPopes DaniPopes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants