Skip to content

feat(net):optimize sync service#5195

Merged
jwrct merged 4 commits intotronprotocol:release_v4.7.2from
jwrct:optimize_sync
May 8, 2023
Merged

feat(net):optimize sync service#5195
jwrct merged 4 commits intotronprotocol:release_v4.7.2from
jwrct:optimize_sync

Conversation

@jwrct
Copy link
Copy Markdown
Contributor

@jwrct jwrct commented May 5, 2023

What does this PR do?
Optimize the synchronization service to ensure the service running on low-performance nodes.

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 5, 2023

Codecov Report

Merging #5195 (1e3e385) into release_v4.7.2 (afb14ec) will increase coverage by 0.15%.
The diff coverage is 35.29%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@                 Coverage Diff                  @@
##             release_v4.7.2    #5195      +/-   ##
====================================================
+ Coverage             59.13%   59.28%   +0.15%     
- Complexity             8797     8823      +26     
====================================================
  Files                   836      836              
  Lines                 49613    49628      +15     
  Branches               5520     5524       +4     
====================================================
+ Hits                  29338    29424      +86     
+ Misses                17950    17876      -74     
- Partials               2325     2328       +3     
Impacted Files Coverage Δ
common/src/main/java/org/tron/core/Constant.java 50.00% <ø> (ø)
.../src/main/java/org/tron/core/config/args/Args.java 59.80% <28.57%> (+1.40%) ⬆️
...va/org/tron/core/net/service/sync/SyncService.java 59.37% <33.33%> (-0.52%) ⬇️
...ava/org/tron/common/parameter/CommonParameter.java 57.02% <50.00%> (-0.05%) ⬇️
...e/net/messagehandler/ChainInventoryMsgHandler.java 17.80% <50.00%> (+1.14%) ⬆️

... and 20 files with indirect coverage changes

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

@jwrct jwrct merged commit 3fa8c4f into tronprotocol:release_v4.7.2 May 8, 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