Skip to content

refactor(block): reduce max size in da submitter #2720

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

Merged
merged 4 commits into from
Oct 1, 2025
+13 −142
Merged

refactor(block): reduce max size in da submitter #2720

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
bd52066
reduce max size
tac0turtle Oct 1, 2025
2b1683f
Merge branch 'main' into marko/reduce_max
tac0turtle Oct 1, 2025
10621a0
remove other consts
tac0turtle Oct 1, 2025
85d36b1
pass maxbytes down to jsonrpc
tac0turtle Oct 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
91 changes: 0 additions & 91 deletions .github/workflows/check_consts_drift.yml
View file
Open in desktop

This file was deleted.

2 changes: 1 addition & 1 deletion block/internal/submitting/da_submitter.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ const (
initialBackoff = 100 * time.Millisecond
defaultGasPrice = 0.0
defaultGasMultiplier = 1.0
defaultMaxBlobSize = 2 * 1024 * 1024 // 2MB fallback blob size limit
defaultMaxBlobSize = 1.5 * 1024 * 1024 // 1.5MB fallback blob size limit
Copy link
Member

Choose a reason for hiding this comment

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

if we merge this, we need to decrease it here too: https://github.com/evstack/ev-node/blob/main/da/jsonrpc/internal/consts.go#L20

defaultMaxGasPriceClamp = 1000.0
defaultMaxGasMultiplierClamp = 3.0 // must always > 0 to avoid division by zero
)
Expand Down
8 changes: 6 additions & 2 deletions da/jsonrpc/client.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,11 @@ import (
"github.com/rs/zerolog"

"github.com/evstack/ev-node/core/da"
internal "github.com/evstack/ev-node/da/jsonrpc/internal"
)

const (
// DefaultMaxBlobSize is the default maximum blob size in bytes.
DefaultMaxBlobSize = 1.5 * 1024 * 1024 // 1.5MB
)

//go:generate mockgen -destination=mocks/api.go -package=mocks . Module
Expand Down Expand Up @@ -232,7 +236,7 @@ func newClient(ctx context.Context, logger zerolog.Logger, addr string, authHead
var multiCloser multiClientCloser
var client Client
client.DA.Logger = logger
client.DA.MaxBlobSize = uint64(internal.MaxTxSize)
client.DA.MaxBlobSize = uint64(DefaultMaxBlobSize)
client.DA.gasPrice = gasPrice
client.DA.gasMultiplier = gasMultiplier

Expand Down
41 changes: 0 additions & 41 deletions da/jsonrpc/internal/consts.go
View file
Open in desktop

This file was deleted.

Loading