-
Notifications
You must be signed in to change notification settings - Fork 141
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
BlockChain<T>.GetStates() returns significantly slow if the height is too high #189
Labels
bug
Something isn't working
Comments
dahlia
changed the title
BlockChain<T>.GetStates(A) where A is nonexistent returns significantly slow if the height is enough high
BlockChain<T>.GetStates(A) where A is nonexistent returns significantly slow if the height is too high
Apr 10, 2019
dahlia
changed the title
BlockChain<T>.GetStates(A) where A is nonexistent returns significantly slow if the height is too high
BlockChain<T>.GetStates() returns significantly slow if the height is too high
Apr 12, 2019
It seems to happen even if a request address is existent. The current implementation compares two sets (more exactly, intended to be a set but represented as libplanet/Libplanet/Blockchain/BlockChain.cs Line 178 in 65bc8cd
|
dahlia
added a commit
to dahlia/libplanet
that referenced
this issue
Apr 12, 2019
dahlia
added a commit
to dahlia/libplanet
that referenced
this issue
Apr 12, 2019
dahlia
added a commit
to dahlia/libplanet
that referenced
this issue
Apr 12, 2019
dahlia
added a commit
to dahlia/libplanet
that referenced
this issue
Apr 12, 2019
dahlia
added a commit
to dahlia/libplanet
that referenced
this issue
Apr 12, 2019
OnedgeLee
pushed a commit
to OnedgeLee/libplanet
that referenced
this issue
Jan 31, 2023
…netarium#189) * INTERNAL: update configmap-versions.yaml * INTERNAL: update kustomization.yaml
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
BlockChain<T>.GetStates(A)
whereA
is nonexistent (i.e., never used) returns significantly slow if the height is enough high. CurrentlyIStore.GetBlockStates()
returns only a set of updated states, not a complete set of existent accounts. This means thatBlockChain<T>.GetStates(A)
needs to descend to the bottom (i.e., the genesis block) in order to determine ifA
is nonexistent.The text was updated successfully, but these errors were encountered: