Skip to content

fix(net):optimize network log#4991

Merged
jwrct merged 1 commit intotronprotocol:release_v4.7.1from
jwrct:optimize_net_logs
Feb 15, 2023
Merged

fix(net):optimize network log#4991
jwrct merged 1 commit intotronprotocol:release_v4.7.1from
jwrct:optimize_net_logs

Conversation

@jwrct
Copy link
Contributor

@jwrct jwrct commented Feb 14, 2023

What does this PR do?
change the log level of bad message from warn to error
Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link

Codecov Report

Merging #4991 (ab07d2f) into release_v4.7.1 (1450156) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

📣 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.1    #4991      +/-   ##
====================================================
- Coverage             54.76%   54.74%   -0.02%     
+ Complexity             8225     8224       -1     
====================================================
  Files                   840      840              
  Lines                 49638    49641       +3     
  Branches               5447     5448       +1     
====================================================
- Hits                  27183    27175       -8     
- Misses                20318    20328      +10     
- Partials               2137     2138       +1     
Impacted Files Coverage Δ
...in/java/org/tron/core/net/P2pEventHandlerImpl.java 9.17% <0.00%> (-0.26%) ⬇️
...in/java/org/tron/core/vm/PrecompiledContracts.java 70.09% <0.00%> (-0.60%) ⬇️
...n/java/org/tron/core/db2/core/SnapshotManager.java 72.93% <0.00%> (-0.57%) ⬇️

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

@jwrct jwrct merged commit 96b0cfc into tronprotocol:release_v4.7.1 Feb 15, 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