Skip to content
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

[cmd] PeerWatcher: A tools for find and monitor peer on peer to peer network. #2409

Merged
merged 1 commit into from
Apr 14, 2021

Conversation

jolestar
Copy link
Member

No description provided.

@github-actions
Copy link

Benchmark for 4ebf1ed

Click to view benchmark
Test PR Benchmark Master Benchmark %
accumulator_append 1406.4±83.32µs 1417.4±77.31µs -0.78%
block_apply/block_apply_10 387.7±2.56ms 397.0±3.02ms -2.34%
block_apply/block_apply_1000 40.6±0.37s 42.4±0.35s -4.25%
get_with_proof/db_store 44.7±1.09µs 45.4±1.23µs -1.54%
get_with_proof/mem_store 34.8±0.74µs 34.9±0.81µs -0.29%
put_and_commit/db_store/1 623.0±87.00µs 619.6±27.99µs +0.55%
put_and_commit/db_store/10 1811.9±284.60µs 1735.7±130.44µs +4.39%
put_and_commit/db_store/100 10.0±0.57ms 10.1±0.52ms -0.99%
put_and_commit/db_store/5 1135.9±94.59µs 1145.7±87.68µs -0.86%
put_and_commit/db_store/50 5.6±0.22ms 5.8±0.30ms -3.45%
put_and_commit/mem_store/1 75.3±4.01µs 75.9±4.16µs -0.79%
put_and_commit/mem_store/10 665.9±33.20µs 680.3±33.81µs -2.12%
put_and_commit/mem_store/100 6.3±0.24ms 6.3±0.23ms 0.00%
put_and_commit/mem_store/5 341.5±18.76µs 349.4±17.22µs -2.26%
put_and_commit/mem_store/50 3.2±0.13ms 3.2±0.15ms 0.00%
query_block/query_block_in(10)_times(100) 3.3±0.06ms 3.4±0.09ms -2.94%
query_block/query_block_in(10)_times(1000) 32.7±0.87ms 33.7±0.87ms -2.97%
query_block/query_block_in(10)_times(10000) 336.9±7.67ms 339.2±5.18ms -0.68%
query_block/query_block_in(1000)_times(100) 1364.7±20.50µs 1401.7±8.21µs -2.64%
query_block/query_block_in(1000)_times(1000) 13.7±0.21ms 13.9±0.15ms -1.44%
query_block/query_block_in(1000)_times(10000) 136.6±1.64ms 139.8±1.24ms -2.29%
storage_transaction 16.5±0.92ms 17.1±4.58ms -3.51%
vm/transaction_execution/1 395.5±3.44ms 398.0±3.08ms -0.63%
vm/transaction_execution/10 129.2±0.63ms 129.6±0.86ms -0.31%
vm/transaction_execution/20 120.6±1.15ms 121.0±0.92ms -0.33%
vm/transaction_execution/5 157.5±2.51ms 157.5±1.57ms 0.00%
vm/transaction_execution/50 136.4±1.11ms 138.0±2.38ms -1.16%

@codecov
Copy link

codecov bot commented Apr 13, 2021

Codecov Report

Merging #2409 (bb95157) into master (a23a8f7) will increase coverage by 0.10%.
The diff coverage is 7.70%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2409      +/-   ##
==========================================
+ Coverage   33.67%   33.77%   +0.10%     
==========================================
  Files         481      482       +1     
  Lines       41533    41540       +7     
  Branches    18681    18685       +4     
==========================================
+ Hits        13983    14025      +42     
+ Misses      13263    13157     -106     
- Partials    14287    14358      +71     
Flag Coverage Δ
unittests 33.77% <7.70%> (+0.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
network/src/service.rs 31.84% <0.00%> (ø)
network/src/worker.rs 34.38% <0.00%> (ø)
network/api/src/tests.rs 22.06% <6.67%> (-2.13%) ⬇️
cmd/peer-watcher/src/lib.rs 100.00% <100.00%> (ø)
commons/timeout-join-handler/src/lib.rs 40.43% <0.00%> (-2.12%) ⬇️
config/src/lib.rs 27.50% <0.00%> (-1.19%) ⬇️
network-p2p/src/protocol.rs 40.21% <0.00%> (ø)
cmd/starcoin/src/cli_state.rs 47.37% <0.00%> (ø)
vm/vm-runtime/src/starcoin_vm.rs 29.97% <0.00%> (+0.33%) ⬆️
... and 6 more

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 a23a8f7...bb95157. Read the comment docs.

@sanlee42
Copy link
Member

Cargo.lock 好像没有更新

@jolestar jolestar force-pushed the feature/peer_watcher branch from 448ec18 to 3e412ef Compare April 13, 2021 10:40
@jolestar jolestar force-pushed the feature/peer_watcher branch from 3e412ef to bb95157 Compare April 13, 2021 11:22
@github-actions
Copy link

Benchmark for ce01b66

Click to view benchmark
Test PR Benchmark Master Benchmark %
accumulator_append 1276.0±61.07µs 1312.2±87.62µs -2.76%
block_apply/block_apply_10 374.5±2.01ms 376.9±4.47ms -0.64%
block_apply/block_apply_1000 38.2±0.33s 39.5±0.16s -3.29%
get_with_proof/db_store 43.4±0.88µs 43.9±1.21µs -1.14%
get_with_proof/mem_store 33.1±0.69µs 33.9±1.36µs -2.36%
put_and_commit/db_store/1 601.3±23.41µs 594.3±18.77µs +1.18%
put_and_commit/db_store/10 1717.6±667.90µs 1665.2±120.91µs +3.15%
put_and_commit/db_store/100 9.4±0.58ms 9.7±0.42ms -3.09%
put_and_commit/db_store/5 1075.5±90.35µs 1081.1±69.62µs -0.52%
put_and_commit/db_store/50 5.4±0.27ms 5.5±0.21ms -1.82%
put_and_commit/mem_store/1 72.7±3.91µs 73.9±4.40µs -1.62%
put_and_commit/mem_store/10 651.4±33.63µs 667.2±30.15µs -2.37%
put_and_commit/mem_store/100 6.1±0.23ms 6.1±0.31ms 0.00%
put_and_commit/mem_store/5 331.4±17.74µs 341.9±20.69µs -3.07%
put_and_commit/mem_store/50 3.1±0.14ms 3.2±0.13ms -3.13%
query_block/query_block_in(10)_times(100) 3.2±0.06ms 3.3±0.07ms -3.03%
query_block/query_block_in(10)_times(1000) 32.3±0.69ms 32.5±0.95ms -0.62%
query_block/query_block_in(10)_times(10000) 323.9±7.64ms 328.1±7.31ms -1.28%
query_block/query_block_in(1000)_times(100) 1283.5±9.05µs 1352.9±20.33µs -5.13%
query_block/query_block_in(1000)_times(1000) 13.0±0.15ms 13.4±0.18ms -2.99%
query_block/query_block_in(1000)_times(10000) 129.5±1.00ms 133.8±1.70ms -3.21%
storage_transaction 16.0±0.67ms 15.9±0.76ms +0.63%
vm/transaction_execution/1 380.5±3.34ms 386.2±6.95ms -1.48%
vm/transaction_execution/10 122.6±1.53ms 127.0±2.26ms -3.46%
vm/transaction_execution/20 114.4±1.43ms 117.3±1.99ms -2.47%
vm/transaction_execution/5 150.4±2.11ms 153.2±2.91ms -1.83%
vm/transaction_execution/50 129.5±0.95ms 133.0±2.51ms -2.63%

@jolestar jolestar merged commit 47a69cb into master Apr 14, 2021
@jolestar jolestar deleted the feature/peer_watcher branch April 14, 2021 01:20
naughtyvenom pushed a commit to naughtyvenom/starcoin that referenced this pull request Jul 19, 2021
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.

2 participants