Skip to content

Remove trans-cache from checkpoint#4815

Merged
zhang0125 merged 1 commit intodevelopfrom
fix/cache_store
Dec 1, 2022
Merged

Remove trans-cache from checkpoint#4815
zhang0125 merged 1 commit intodevelopfrom
fix/cache_store

Conversation

@zhang0125
Copy link
Copy Markdown
Contributor

What does this PR do?

Remove trans-cache from checkpoint

Why are these changes required?

Trans-cache is deprecated

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #4815 (fc6a835) into develop (82c40b4) will decrease coverage by 0.13%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #4815      +/-   ##
=============================================
- Coverage      54.22%   54.09%   -0.14%     
+ Complexity      8048     8028      -20     
=============================================
  Files            833      833              
  Lines          47707    47709       +2     
  Branches        5183     5184       +1     
=============================================
- Hits           25871    25806      -65     
- Misses         19780    19847      +67     
  Partials        2056     2056              
Impacted Files Coverage Δ
...n/java/org/tron/core/db2/core/SnapshotManager.java 74.20% <100.00%> (+0.15%) ⬆️
...n/common/net/udp/message/discover/PongMessage.java 0.00% <0.00%> (-90.00%) ⬇️
...src/main/java/org/tron/core/db/PendingManager.java 34.88% <0.00%> (-16.28%) ⬇️
.../java/org/tron/core/consensus/BlockHandleImpl.java 65.00% <0.00%> (-15.00%) ⬇️
...tron/common/overlay/discover/node/NodeHandler.java 51.56% <0.00%> (-10.94%) ⬇️
...udp/message/discover/DiscoverMessageInspector.java 21.62% <0.00%> (-8.11%) ⬇️
.../java/org/tron/common/net/udp/message/Message.java 56.52% <0.00%> (-4.35%) ⬇️
.../tron/common/overlay/discover/table/NodeTable.java 90.38% <0.00%> (-3.85%) ⬇️
...rc/main/java/org/tron/consensus/dpos/DposTask.java 88.70% <0.00%> (-3.23%) ⬇️
...ay/discover/node/statistics/MessageStatistics.java 91.24% <0.00%> (-2.92%) ⬇️
... and 3 more

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

@zhang0125 zhang0125 merged commit 8f78620 into develop Dec 1, 2022
@zhang0125 zhang0125 deleted the fix/cache_store branch December 1, 2022 02:46
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.

3 participants