dymensionxyz / dymint

Sequencing Engine for Dymension RollApps
Other
97 stars 67 forks source link

Commit size + Block size could be greater than batch size #885

Closed omritoptix closed 3 months ago

omritoptix commented 4 months ago

The issue here is that though we take for the block size the minimum between the batch size and the block size consensus params, in those lines we also append the commit to the batch (+ the overhead of the struct of the batch itself) so in some cases the batch won't get populated with any blocks (as it may be that the first block in the batch + commit > max batch size) which would lead to inability to submit batches.