Skip to content

dnm: experiment: include all transactions in block proposal #12069

Closed
Maddiaa0 wants to merge 7 commits intomasterfrom
md/exp-txs-in-block-proposal
Closed

dnm: experiment: include all transactions in block proposal #12069
Maddiaa0 wants to merge 7 commits intomasterfrom
md/exp-txs-in-block-proposal

Conversation

@Maddiaa0
Copy link
Member

@Maddiaa0 Maddiaa0 commented Feb 18, 2025

Overview

Modifies the block proposal to include all of its transactions, such that they are all propagated together. Can use this pr to benchmark block proposals and see how large they get

Thoughts

  • this could simplify many things - we dont need to worry about attestors not having transactions for a proposal, instead they are all included
  • it simplifies dealing with private mempools
  • issue - it makes block proposals MASSIVE, but as discussed with alexey, might just be best to innovate a bit around chunking this and propagating it in a different way than default gossip

linked to #12055

Copy link
Member Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Maddiaa0 Maddiaa0 force-pushed the md/exp-txs-in-block-proposal branch from 6ce7eba to 626ba48 Compare February 18, 2025 13:07
Base automatically changed from md/02-17-chore_cleanup_libp2p_logger to master February 18, 2025 13:28
@Maddiaa0
Copy link
Member Author

stale experiment

@Maddiaa0 Maddiaa0 closed this Mar 26, 2025
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.

1 participant