Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Re-include missed txs into our blocks #9

Open
asonnino opened this issue Jun 17, 2021 · 1 comment
Open

Re-include missed txs into our blocks #9

asonnino opened this issue Jun 17, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@asonnino
Copy link
Contributor

Currently, we never do it. But it might be the case that the upper layer consensus declined our block and thus we need to re-include it TXs. We need to think about the API with the consensus layer - it should tell as which blocks we can move to cold storage and which we need to retry.

We currently re-include digests until they appear in certified header. However, a certified header might still not get into the DAG. So need to think of a more accurate condition to stop re-include digests.

@asonnino asonnino added the enhancement New feature or request label Jun 17, 2021
@asonnino asonnino changed the title When and how should we re-include TX in our blocks? Re-include missed txs into our blocks Jun 17, 2021
@LefKok
Copy link

LefKok commented Jun 28, 2021

Otherwise, add weak links in hope of including the slow blocks before GC

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants