You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
// block_data is an array of transactions that will be included in a block,
// sent to the app for possible modifications.
// applications can not exceed the size of the data passed to it.
tendermint.types.Datablock_data=1;
// If an application decides to populate block_data with extra information, they can not exceed this value.
int64block_data_size=2;
Problem
I think these comments aren't clear.
// block_data is an array of transactions that will be included in a block, sent to the app for possible modifications.
block_data is an array of candidate transactions that may be included in a block. The array is sent to the application to filter out any invalid transactions and re-order all SDK transactions before blob transactions. Note the transactions themselves aren't modified.
// applications can not exceed the size of the data passed to it.
IDK what this means because the application does in fact return a BlockData with a size greater than the data passed in. The BlockData that the application returns includes a SquareSize and hash (data root).
// If an application decides to populate block_data with extra information, they can not exceed this value.
IMO this should clarify that the total BlockData can't exceed BlockDataSize. The way it's written currently made me think that just the extra information can no exceed BlockDataSize.
Proposal
Update the comments to make them clearer.
Note: the CometBFT RequestPrepareProposal type looks a bit different from ours so we can't copy their comments.
The text was updated successfully, but these errors were encountered:
Context
See Slack thread
celestia-core/proto/tendermint/abci/types.proto
Lines 129 to 135 in fb9d656
Problem
I think these comments aren't clear.
block_data
is an array of candidate transactions that may be included in a block. The array is sent to the application to filter out any invalid transactions and re-order all SDK transactions before blob transactions. Note the transactions themselves aren't modified.IDK what this means because the application does in fact return a
BlockData
with a size greater than the data passed in. The BlockData that the application returns includes a SquareSize and hash (data root).IMO this should clarify that the total
BlockData
can't exceedBlockDataSize
. The way it's written currently made me think that just the extra information can no exceedBlockDataSize
.Proposal
Update the comments to make them clearer.
Note: the CometBFT RequestPrepareProposal type looks a bit different from ours so we can't copy their comments.
The text was updated successfully, but these errors were encountered: