Skip to content

Ln/replace best chain with block index #1413

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.

Sign up for GitHub

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

Jump to bottom
Merged
lazynina merged 134 commits into from
May 1, 2025
Merged

Ln/replace best chain with block index #1413

lazynina merged 134 commits into from
May 1, 2025

Conversation

@lazynina
Copy link
Member

@lazynina lazynina commented Oct 3, 2024

No description provided.

lazynina and others added 30 commits September 23, 2024 18:50
@lazynina
start by refactoring block index into a read-through cache
fc9a0d3
@lazynina
best chain and header chain with read through cache
f3e8e4c
@lazynina
check bestHeaderChain.ChainMap ONLY when processing pos headers
a2e5b4b
@lazynina
updates for locating headers and faster cache usage
ff6ee24
@lazynina
more fixes
71b1eaa
@lazynina
more fixes
17a56bd
@lazynina
more fix, specifically around .Ancestor
d153927
@lazynina
bump test timeout
47e1958
@lazynina
load all blocks after committed tip in block index
5e5c0c1
@lazynina
fix GetBlockNodeByHashAndHeight - no need for checking height > maxHe…
9a32c26 
…ight
@lazynina
bump timeout to 60m
5359be8
@lazynina
try hashicorp lru cache
89a845e
@lazynina
Fix up some more tests and remove logging
4f7ff12
@lazynina
fix string slice workaround, fix calc next difficulty
722dcc0
@lazynina
use best header chain instead of best chain for calc next difficulty …
1941487 
…target
@lazynina
fix tests
627e882
@lazynina
refactor to just use a block hash for best block chain and best heade…
a469c3a 
…r chain representation
@lazynina
move block index migration to NewBlockchain instead of NewServer, add…
c6ad24a 
… setting committed status for PoW nodes, remove timeout adjustment in test.Dockerfile
@lazynina
merge main into branch
216fd31
@lazynina
Merge branch 'ln/read-through-cache-for-block-index-and-best-chain' i…
1f80f19 
…nto ln/replace-best-chain-with-block-index
@lazynina
handle case where we haven't hit PoS cutover yet in block index migra…
8b73b1c 
…tion
@lazynina
cleanup
7ac67bd
@lazynina
do some sampling for is fully stored and check archival mode
7675765
@lazynina
fix get blocks to store
148bad3
@lazynina
cleanup
8aa6a5b
@lazynina
make a bunch of methods on block index private and expose block index…
53b2d29 
… to backend
@lazynina
don't bother healing orphan pointers if we're syncing
91e22ff
@lazynina
refactor to reduce number of times we have to call hash and don't bot…
9abe238 
…her healing orphan pointers during syncing
@lazynina
upgrade cloudflare/circl
23991c4
@lazynina
Ln/replace best chain plus replace old lru cache (#1414)
8adc624 
* replace decred lru with hashicorp lru

* fix usage of delete -> remove

* fix calc next difficulty target issue

* upgrade cloudflare/circl
lazynina added 28 commits January 8, 2025 16:46
@lazynina
go mod tidy
36b605e
@lazynina
reduce size of walkback in initChain to 6 hours
0a9499e
@lazynina
update GetSpendableDeSoBalanceNanosForPublicKey
11773dd
@lazynina
fix GetSpendableUtxosForPublicKey
587ac72
@lazynina
revert changes for get spendable balance and utxos. update get spenda…
05a2ae3 
…ble balance to not use Header.PrevBlockHash
@lazynina
Merge branch 'main' into ln/replace-best-chain-with-block-index
d40b2f1
@lazynina
update some error messages
f963570
@lazynina
switch HandleInv to respond based on isSyncing function rather than c…
8d8bb6d 
…hain state = fully current
@lazynina
skipping invs for peer - log level 3
db8469b
@lazynina
loop over locators in GetHeadersForLocatorAndStopHash
f2bf0d7
@lazynina
switch handleInv to only when fully current
ca638f9
@lazynina
skip update after connect/disconnect block hooks in legacy mempool if…
ea85d4e 
… block is pos block
@lazynina
update GetBlockNodesToFetch to use GetParent
e73d998
@lazynina
Clean up
ca1d91b
@lazynina
revert HandleInv to use isSyncing instead of checking for SyncStateFu…
301891f 
…llyCurrent
@lazynina
Merge branch 'main' into ln/replace-best-chain-with-block-index
9f25d89
@lazynina
Merge branch 'main' into ln/replace-best-chain-with-block-index
0654f4c
@lazynina
fix IsFullyStored
04375be
@lazynina
Revert LatestHeaderLocator changes
d0428c2
@lazynina
more closely replicate old LatestLocator function, also use this func…
d748ca6 
…tion in new HeaderLocatorWithNodeHashAndHeight which replaces HeaderLocatorWithNodeHash
@lazynina
use DerivedKeyEntry.Copy in utxo view copy
bc87a14
@lazynina
add optimization in get block nodes to fetch
dfcdb14
@lazynina
Merge branch 'main' into ln/replace-best-chain-with-block-index
65e6244
@lazynina
Merge branch 'main' into ln/replace-best-chain-with-block-index
0ac4385
@lazynina
resolve merge conflicts
837ea3b
@lazynina
add validatorPK in error about duplicate validator public key
e07c942
@lazynina
use true for isHeaderChain in LatestLocator
38029eb
@lazynina
reverse slice at the end of GetHeadersForLocatorAndStopHash instead o…
66ddf70 
…f in each iteration
@lazynina lazynina merged commit 2a55a58 into main May 1, 2025
3 checks passed
@lazynina lazynina deleted the ln/replace-best-chain-with-block-index branch May 1, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@diamondhands0 diamondhands0 diamondhands0 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lazynina @diamondhands0