Skip to content

refactor(log): downgrade certain log level in some files#5121

Merged
halibobo1205 merged 1 commit intotronprotocol:release_v4.7.2from
halibobo1205:feature/log_opt
Apr 11, 2023
Merged

refactor(log): downgrade certain log level in some files#5121
halibobo1205 merged 1 commit intotronprotocol:release_v4.7.2from
halibobo1205:feature/log_opt

Conversation

@halibobo1205
Copy link
Copy Markdown
Collaborator

@halibobo1205 halibobo1205 commented Apr 10, 2023

What does this PR do?

downgrade certain log level for parameter verification
Why are these changes required?
ERROR is not necessary for parameter verification

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@halibobo1205 halibobo1205 force-pushed the feature/log_opt branch 2 times, most recently from 277dbee to 1852844 Compare April 10, 2023 09:11
@halibobo1205 halibobo1205 changed the title refactor(log): downgrade log refactor(log): downgrade certain log level in some files Apr 10, 2023
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #5121 (36e9e10) into release_v4.7.2 (c0955d7) will increase coverage by 0.04%.
The diff coverage is 14.28%.

📣 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    #5121      +/-   ##
====================================================
+ Coverage             55.77%   55.81%   +0.04%     
- Complexity             8346     8355       +9     
====================================================
  Files                   836      836              
  Lines                 49562    49564       +2     
  Branches               5512     5512              
====================================================
+ Hits                  27643    27665      +22     
+ Misses                19703    19688      -15     
+ Partials               2216     2211       -5     
Impacted Files Coverage Δ
...java/org/tron/core/capsule/TransactionCapsule.java 52.66% <0.00%> (ø)
...mework/src/main/java/org/tron/core/db/Manager.java 52.99% <0.00%> (-0.08%) ⬇️
...rc/main/java/org/tron/core/services/http/Util.java 34.72% <0.00%> (ø)
...org/tron/core/services/jsonrpc/JsonRpcApiUtil.java 24.23% <0.00%> (ø)
framework/src/main/java/org/tron/core/Wallet.java 39.66% <22.22%> (+<0.01%) ⬆️

... and 10 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 b4b1444 into tronprotocol:release_v4.7.2 Apr 11, 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