Skip to content

add transaction data #3200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 8, 2020

Conversation

DorianRust
Copy link
Contributor

What does this PR do?
add data filed in the transaction log trigger
Why are these changes required?
add data filed in the transaction log trigger

This PR has been tested by:

  • Unit Tests
  • Manual Testing

@codecov-commenter
Copy link

codecov-commenter commented Jun 5, 2020

Codecov Report

Merging #3200 into release_vm_shielded_4.0 will increase coverage by 0.06%.
The diff coverage is 0.00%.

Impacted file tree graph

@@                      Coverage Diff                      @@
##             release_vm_shielded_4.0    #3200      +/-   ##
=============================================================
+ Coverage                      31.31%   31.37%   +0.06%     
- Complexity                      2023     2031       +8     
=============================================================
  Files                            393      393              
  Lines                          20488    20490       +2     
  Branches                        2339     2339              
=============================================================
+ Hits                            6416     6429      +13     
+ Misses                         13330    13324       -6     
+ Partials                         742      737       -5     
Impacted Files Coverage Δ Complexity Δ
...gsfilter/capsule/TransactionLogTriggerCapsule.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...tron/common/overlay/discover/node/NodeHandler.java 51.61% <0.00%> (-1.62%) 18.00% <0.00%> (-2.00%)
...g/tron/common/overlay/discover/DiscoverServer.java 82.75% <0.00%> (+1.72%) 8.00% <0.00%> (+1.00%)
...ava/org/tron/common/overlay/server/P2pHandler.java 34.04% <0.00%> (+2.12%) 10.00% <0.00%> (+1.00%)
.../tron/common/overlay/discover/table/NodeEntry.java 46.51% <0.00%> (+2.32%) 7.00% <0.00%> (+1.00%)
.../tron/common/overlay/discover/table/NodeTable.java 38.46% <0.00%> (+3.07%) 11.00% <0.00%> (+1.00%)
...mon/logsfilter/nativequeue/NativeMessageQueue.java 83.33% <0.00%> (+6.66%) 11.00% <0.00%> (+2.00%)
...g/tron/common/overlay/server/MessageRoundtrip.java 92.85% <0.00%> (+7.14%) 7.00% <0.00%> (+1.00%)
...a/org/tron/common/overlay/server/MessageQueue.java 58.33% <0.00%> (+7.29%) 19.00% <0.00%> (+2.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d303c1...fdda805. Read the comment docs.

@DorianRust DorianRust merged commit bb7d227 into release_vm_shielded_4.0 Jun 8, 2020
@ethan1844 ethan1844 deleted the addTransactionData2shielded_4.0 branch March 30, 2022 10:27
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