Skip to content

feat(db): optimize bloom filter for transaction cache#5263

Merged
halibobo1205 merged 1 commit intotronprotocol:release_v4.7.2from
halibobo1205:fix/bloom_filters_init_opt
Jun 7, 2023
Merged

feat(db): optimize bloom filter for transaction cache#5263
halibobo1205 merged 1 commit intotronprotocol:release_v4.7.2from
halibobo1205:fix/bloom_filters_init_opt

Conversation

@halibobo1205
Copy link
Copy Markdown
Collaborator

What does this PR do?
make bloom filter initialization after checkpoint recovery
Why are these changes required?
optimize bloom filter initialization for transaction cache
This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #5263 (47f659c) into release_v4.7.2 (0a145f4) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@                 Coverage Diff                  @@
##             release_v4.7.2    #5263      +/-   ##
====================================================
+ Coverage             60.50%   60.52%   +0.02%     
- Complexity             9106     9112       +6     
====================================================
  Files                   839      839              
  Lines                 49920    49924       +4     
  Branches               5559     5559              
====================================================
+ Hits                  30204    30217      +13     
+ Misses                17340    17330      -10     
- Partials               2376     2377       +1     
Impacted Files Coverage Δ
.../main/java/org/tron/core/db2/common/TxCacheDB.java 83.49% <ø> (-0.16%) ⬇️
...n/java/org/tron/core/db/TronStoreWithRevoking.java 84.81% <100.00%> (+0.19%) ⬆️
...n/java/org/tron/core/db2/core/SnapshotManager.java 74.02% <100.00%> (+0.07%) ⬆️
...mework/src/main/java/org/tron/core/db/Manager.java 57.27% <100.00%> (+0.70%) ⬆️
...c/main/java/org/tron/core/db/TransactionCache.java 100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@halibobo1205 halibobo1205 merged commit 0efe71c into tronprotocol:release_v4.7.2 Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants