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
one simple change we do for all forms of block prop (including mempools), is to utilize a dynamic queue.
For the existing consensus reactor, we queue block parts to be send for each peer every MaxGossipDelay. It's very possible to receive a block part from a peer that we have queued but have yet to pass through to the tcp buffer. When that's the case, it would be more optimal to not send that block part. If we had the ability to remove block parts from they queue, then nodes could do this as they receive block parts from their peers.
For mempools, the highest priority txs can be gossiped first, not unlike #1069. However, from a discussion w/ @cmwaters, we could move beyond #1069, and we can define priority differently. Instead of only basing this on gas price, we can always feed peers txs that are included in high priority VACs or compact blocks.
we should split this issue up into two: the ability to remove things from the queue and the ability to sort. We only need the ability to remove for push based, and the ability to sort for pull based
one simple change we do for all forms of block prop (including mempools), is to utilize a dynamic queue.
For the existing consensus reactor, we queue block parts to be send for each peer every
MaxGossipDelay
. It's very possible to receive a block part from a peer that we have queued but have yet to pass through to the tcp buffer. When that's the case, it would be more optimal to not send that block part. If we had the ability to remove block parts from they queue, then nodes could do this as they receive block parts from their peers.For mempools, the highest priority txs can be gossiped first, not unlike #1069. However, from a discussion w/ @cmwaters, we could move beyond #1069, and we can define priority differently. Instead of only basing this on gas price, we can always feed peers txs that are included in high priority VACs or compact blocks.
an example implementation was done in the big blonks testnet here https://github.com/celestiaorg/celestia-core/blob/evan/pipeline-cat-hack/mempool/cat/sorted_queue.go
The text was updated successfully, but these errors were encountered: