8000 Fix APM block-zero context gap · Issue #339 · VeriBlock/nodecore · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix APM block-zero context gap #339

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Warchant opened this issue Nov 19, 2021 · 0 comments
Open

Fix APM block-zero context gap #339

Warchant opened this issue Nov 19, 2021 · 0 comments
Labels
apm bug Something isn't working prio:low

Comments

@Warchant
Copy link
Collaborator

Test case that shows the problem: https://github.com/VeriBlock/alt-integration-cpp/blob/ffb90e4d8a2c321dc011a3b73f421f55fce67197/test/pop/mempool_test.cpp#L490
In short:
VBK8 contains VTB0 with BTC[1..(6)]
VBK9 contains VTB1 with BTC[8..(9)]
VBK10 contains VTB2 with BTC[7..(11)]
VTB1 can not be connected to active chain, because its context starts at 8, but last known is 6. It does connect only to FUTURE block, because it brings missing BTC7.
In presence of this scenario, the fix is to search for another VTB which brings BTC7, but whose containing block is <=VBK9, and likely to be >= VBK8

Similar to context gap issue, but worth noting difference between (A) initially starting new chain, (B) existing chain stops for a few days and cannot fill the gap.

Related to these other issues:
ALT-646
NCS-339
ALT-706

@Warchant Warchant moved this to Backlog in Proof-of-Proof Nov 19, 2021
@Warchant Warchant moved this from Backlog to Triaged in Proof-of-Proof Nov 19, 2021
@Warchant Warchant added the bug Something isn't working label Nov 19, 2021
@Warchant Warchant self-assigned this Nov 19, 2021
@Warchant Warchant added the apm label Nov 19, 2021
@Warchant Warchant moved this from Triaged to In Progress in Proof-of-Proof Nov 19, 2021
@Warchant Warchant moved this from In Progress to Triaged in Proof-of-Proof Nov 29, 2021
@Warchant Warchant moved this from Triaged to Backlog in Proof-of-Proof Feb 8, 2022
@Warchant Warchant removed their assignment Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm bug Something isn't working prio:low
Projects
Status: Backlog
Development

No branches or pull requests

1 participant
0