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

[tools] Improve airdrop tools #3532

Merged
merged 2 commits into from
Jul 14, 2022
Merged

[tools] Improve airdrop tools #3532

merged 2 commits into from
Jul 14, 2022

Conversation

jolestar
Copy link
Member

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

提高解析文件的容错能力

Other information

@codecov
Copy link

codecov bot commented Jul 14, 2022

Codecov Report

Merging #3532 (174aca6) into master (521eabd) will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3532      +/-   ##
==========================================
- Coverage   29.30%   29.21%   -0.08%     
==========================================
  Files         589      589              
  Lines       49739    49739              
  Branches    23404    23404              
==========================================
- Hits        14571    14528      -43     
+ Misses      20946    20862      -84     
- Partials    14222    14349     +127     
Flag Coverage Δ
unittests 29.21% <ø> (-0.08%) ⬇️

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

Impacted Files Coverage Δ
rpc/server/src/module/mod.rs 0.00% <0.00%> (-16.66%) ⬇️
chain/open-block/src/lib.rs 31.79% <0.00%> (-9.27%) ⬇️
vm/types/src/on_chain_config/mod.rs 41.10% <0.00%> (-8.21%) ⬇️
vm/types/src/transaction_metadata.rs 55.23% <0.00%> (-7.46%) ⬇️
vm/types/src/transaction/error.rs 3.58% <0.00%> (-7.14%) ⬇️
executor/src/block_executor.rs 25.59% <0.00%> (-6.97%) ⬇️
commons/accumulator/src/tree_store/mock.rs 28.58% <0.00%> (-2.85%) ⬇️
vm/vm-runtime/src/starcoin_vm.rs 36.41% <0.00%> (-2.65%) ⬇️
network-p2p/src/peer_info.rs 43.53% <0.00%> (-2.59%) ⬇️
sync/src/block_connector/test_write_block_chain.rs 31.79% <0.00%> (-2.32%) ⬇️
... and 70 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 521eabd...174aca6. Read the comment docs.

@github-actions
Copy link

Benchmark for 7ef4cba

Click to view benchmark
Test Base PR %
accumulator_append 594.2±13.39µs 593.7±11.01µs -0.08%
block_apply/block_apply_10 265.0±1.71ms 265.6±0.38ms +0.23%
block_apply/block_apply_1000 27.1±0.01s 27.0±0.03s -0.37%
get_with_proof/db_store 36.2±0.44µs 36.3±0.24µs +0.28%
get_with_proof/mem_store 31.2±0.19µs 31.6±0.45µs +1.28%
put_and_commit/db_store/1 95.0±4.84µs 95.3±4.86µs +0.32%
put_and_commit/db_store/10 853.5±39.49µs 854.7±38.67µs +0.14%
put_and_commit/db_store/100 7.3±0.32ms 7.3±0.33ms 0.00%
put_and_commit/db_store/5 436.2±21.93µs 435.9±21.52µs -0.07%
put_and_commit/db_store/50 3.8±0.16ms 3.8±0.17ms 0.00%
put_and_commit/mem_store/1 61.7±5.64µs 61.6±5.67µs -0.16%
put_and_commit/mem_store/10 575.7±46.61µs 575.4±48.46µs -0.05%
put_and_commit/mem_store/100 5.6±0.43ms 5.7±0.82ms +1.79%
put_and_commit/mem_store/5 291.5±26.46µs 292.1±25.49µs +0.21%
put_and_commit/mem_store/50 2.8±0.19ms 2.8±0.18ms 0.00%
query_block/query_block_in(10)_times(100) 3.9±0.07ms 3.9±0.06ms 0.00%
query_block/query_block_in(10)_times(1000) 39.0±0.52ms 38.3±1.02ms -1.79%
query_block/query_block_in(10)_times(10000) 385.8±8.43ms 388.9±8.97ms +0.80%
query_block/query_block_in(1000)_times(100) 966.7±5.65µs 965.8±7.40µs -0.09%
query_block/query_block_in(1000)_times(1000) 9.6±0.09ms 9.8±0.38ms +2.08%
query_block/query_block_in(1000)_times(10000) 95.8±0.62ms 96.5±0.67ms +0.73%
storage_transaction 1072.3±325.49µs 1063.2±296.76µs -0.85%
vm/transaction_execution/1 312.9±3.55ms 313.5±1.58ms +0.19%
vm/transaction_execution/10 98.1±0.23ms 98.4±0.16ms +0.31%
vm/transaction_execution/20 91.7±0.21ms 91.7±0.09ms 0.00%
vm/transaction_execution/5 119.8±0.21ms 120.0±0.15ms +0.17%
vm/transaction_execution/50 105.9±0.24ms 106.1±0.29ms +0.19%

@github-actions
Copy link

Benchmark for 643c55e

Click to view benchmark
Test Base PR %
accumulator_append 601.5±14.43µs 594.4±11.59µs -1.18%
block_apply/block_apply_10 266.0±1.43ms 265.5±0.60ms -0.19%
block_apply/block_apply_1000 27.2±0.03s 27.2±0.03s 0.00%
get_with_proof/db_store 36.2±0.21µs 36.0±0.16µs -0.55%
get_with_proof/mem_store 31.3±0.18µs 31.3±0.20µs 0.00%
put_and_commit/db_store/1 96.2±4.97µs 96.4±7.75µs +0.21%
put_and_commit/db_store/10 852.8±39.03µs 859.9±40.11µs +0.83%
put_and_commit/db_store/100 7.4±0.30ms 7.3±0.32ms -1.35%
put_and_commit/db_store/5 440.5±22.82µs 441.6±22.81µs +0.25%
put_and_commit/db_store/50 3.8±0.19ms 3.8±0.18ms 0.00%
put_and_commit/mem_store/1 62.2±5.63µs 61.7±5.60µs -0.80%
put_and_commit/mem_store/10 577.1±46.87µs 579.7±47.04µs +0.45%
put_and_commit/mem_store/100 5.7±0.83ms 5.6±0.34ms -1.75%
put_and_commit/mem_store/5 293.1±24.90µs 290.2±24.59µs -0.99%
put_and_commit/mem_store/50 2.8±0.18ms 2.8±0.18ms 0.00%
query_block/query_block_in(10)_times(100) 3.9±0.10ms 3.8±0.12ms -2.56%
query_block/query_block_in(10)_times(1000) 39.1±0.74ms 39.1±0.87ms 0.00%
query_block/query_block_in(10)_times(10000) 388.9±5.53ms 391.8±4.46ms +0.75%
query_block/query_block_in(1000)_times(100) 980.3±13.26µs 963.7±5.94µs -1.69%
query_block/query_block_in(1000)_times(1000) 9.8±0.05ms 9.6±0.05ms -2.04%
query_block/query_block_in(1000)_times(10000) 97.9±1.00ms 95.7±0.35ms -2.25%
storage_transaction 1048.5±286.37µs 1048.3±281.38µs -0.02%
vm/transaction_execution/1 312.2±0.49ms 313.6±2.27ms +0.45%
vm/transaction_execution/10 98.5±0.48ms 98.5±0.31ms 0.00%
vm/transaction_execution/20 92.0±0.91ms 92.2±1.14ms +0.22%
vm/transaction_execution/5 120.2±0.65ms 120.2±0.27ms 0.00%
vm/transaction_execution/50 106.7±2.03ms 106.2±0.25ms -0.47%

@nkysg nkysg merged commit 5c28612 into master Jul 14, 2022
@nkysg nkysg deleted the airdrop branch July 14, 2022 10:16
jiangying000 pushed a commit to jiangying000/starcoin that referenced this pull request Jul 22, 2022
* [tools] Improve airdrop tools

* fixup
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