-
Notifications
You must be signed in to change notification settings - Fork 112
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
Blockstore use clean-up #1122
Blockstore use clean-up #1122
Commits on Mar 6, 2020
-
Add isBlockStored method to the repository
This is a step towards limiting access to the block store. The long term goal is to allow only the repository to retrieve block data from the block store and pass it further.
Configuration menu - View commit details
-
Copy full SHA for a4609e4 - Browse repository at this point
Copy the full SHA a4609e4View commit details -
Move block store correction code in the repository
As a result DBUtils no longer accesses the block store directly.
Configuration menu - View commit details
-
Copy full SHA for 9b2e84b - Browse repository at this point
Copy the full SHA 9b2e84bView commit details -
Update messages in DBUtils to use a Logger
The messages now give a better indications of when an error occured.
Configuration menu - View commit details
-
Copy full SHA for 5e01106 - Browse repository at this point
Copy the full SHA 5e01106View commit details -
Configuration menu - View commit details
-
Copy full SHA for e070962 - Browse repository at this point
Copy the full SHA e070962View commit details -
Configuration menu - View commit details
-
Copy full SHA for f52e93b - Browse repository at this point
Copy the full SHA f52e93bView commit details -
Call getTotalDifficultyForHash from repository
Reduces access to the block store by forwarding calls to the blockchain and repository. Also removed the unnecessary hash wrapping.
Configuration menu - View commit details
-
Copy full SHA for f18fbab - Browse repository at this point
Copy the full SHA f18fbabView commit details -
Move loadBlockchain out of AionHub
Having all the functionality to load the blockchain inside the hub requires exposing data elements that would otherwise be kept private. Additionally, the method becomes easier to read when directly making use of the blockchain internal attributes.
Configuration menu - View commit details
-
Copy full SHA for 4526b2f - Browse repository at this point
Copy the full SHA 4526b2fView commit details -
Configuration menu - View commit details
-
Copy full SHA for ba84ce9 - Browse repository at this point
Copy the full SHA ba84ce9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 679c185 - Browse repository at this point
Copy the full SHA 679c185View commit details -
Remove StandaloneBlockchain.setBlockNumber method
The method is no longer used outside the test that checks its functionality. It was a hacky way of creating large chains for testing that can be replaced with the chain generation functionality from BlockchainTestUtils.
Configuration menu - View commit details
-
Copy full SHA for c0bf3dd - Browse repository at this point
Copy the full SHA c0bf3ddView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0282a4d - Browse repository at this point
Copy the full SHA 0282a4dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 44a7108 - Browse repository at this point
Copy the full SHA 44a7108View commit details -
Remove getBlockStore from the blockchain
Access to the block store must go though the repository.
Configuration menu - View commit details
-
Copy full SHA for 50ea22d - Browse repository at this point
Copy the full SHA 50ea22dView commit details