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
Build Block Blooms Database (2.2.0) #1384
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You missed these commits:
240dde7
to
9e739b2
Compare
pipeline:run |
Done
|
cbd7cab
to
4f4631f
Compare
pipeline: run |
4f4631f
to
5809896
Compare
Already done |
06c311c
to
6443014
Compare
…oom method to addBlocksBloom; rename blocksBloom private variable to blocksBloomCache
…g visibility for testing; use HashMap instead of ConcurrentHashMap
…esent in 2.2.0-rc)
6443014
to
11dc6b8
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs 87.2% Coverage The version of Java (1.8.0_275) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 11. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
pipeline: run |
fit: 2.2-rc
Description
Using a new configuration option, the build and save of block blooms data could be executed within the normal operation of the node. This data is used to retrieve log information efficiently.
Motivation and Context
Usually, this block blooms information is used only in nodes that has JSON RPC open, and log query operation enabled. And with the first query, the data is build and saved. But even with this feature, the operation could take many minutes and resources from the node.
So, this new code allow the build of the same data along the blockchain node normal processing, ie, during long synchronization or normal synchronization. To do this, a new internal service was added, and it is activated by configuration (the default value is not activated),
How Has This Been Tested?
It has code tests, and also was tested running a long sync against testnet network,
Types of changes
Checklist: