Skip to content

Commit

Permalink
Increase MaxInputSize to 16MB in op-alt-da
Browse files Browse the repository at this point in the history
On `celo10` branch I've observed in the batcher:

```
{"t":"2024-10-17T15:24:33.473665645Z","lvl":"crit","msg":"Application failed","message":"failed to setup: failed to init channel config: max frame size 999999 exceeds altDA max input size 130672"}
```

With previous `celo` versions this size limit was not enforced, and we increased the max frame size to 16MB in the derivation pipeline ([PR](#228)).
  • Loading branch information
jcortejoso committed Nov 18, 2024
1 parent 9a5acf0 commit 10efb86
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion op-alt-da/params.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,10 @@ package altda

// MaxInputSize ensures the canonical chain cannot include input batches too large to
// challenge in the Data Availability Challenge contract. Value in number of bytes.
// For EigenDA, batch size is not a limiter for Data Availability Challenge contract,
// so this is set to 16MB (current EigenDA batch size limit)
// This value can only be changed in a hard fork.
const MaxInputSize = 130672
const MaxInputSize = 16_000_000

// TxDataVersion1 is the version number for batcher transactions containing
// altDA commitments. It should not collide with DerivationVersion which is still
Expand Down

0 comments on commit 10efb86

Please sign in to comment.