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

[release]Prepare release v1.12.2 #3687

Merged
merged 1 commit into from
Aug 25, 2022
Merged

[release]Prepare release v1.12.2 #3687

merged 1 commit into from
Aug 25, 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 Aug 25, 2022

Codecov Report

Merging #3687 (6fd6e71) into master (c6634ad) will increase coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3687      +/-   ##
==========================================
+ Coverage   28.50%   28.56%   +0.06%     
==========================================
  Files         588      588              
  Lines       50411    50411              
  Branches    23657    23657              
==========================================
+ Hits        14364    14394      +30     
+ Misses      22242    22126     -116     
- Partials    13805    13891      +86     
Flag Coverage Δ
unittests 28.56% <ø> (+0.06%) ⬆️

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

Impacted Files Coverage Δ
consensus/src/consensus.rs 39.69% <0.00%> (-15.87%) ⬇️
sync/src/block_connector/test_write_block_chain.rs 21.71% <0.00%> (-6.20%) ⬇️
vm/natives/src/token.rs 22.23% <0.00%> (-4.44%) ⬇️
commons/timeout-join-handler/src/lib.rs 40.43% <0.00%> (-4.25%) ⬇️
executor/src/executor.rs 45.84% <0.00%> (-4.16%) ⬇️
network-p2p/src/peer_info.rs 45.08% <0.00%> (-3.10%) ⬇️
miner/src/generate_block_event_pacemaker.rs 51.17% <0.00%> (-2.32%) ⬇️
...mmons/accumulator/src/inmemory/accumulator_test.rs 40.91% <0.00%> (-2.27%) ⬇️
commons/stream-task/src/collector.rs 58.17% <0.00%> (-2.04%) ⬇️
vm/vm-runtime/src/data_cache.rs 66.00% <0.00%> (-2.00%) ⬇️
... and 73 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 c6634ad...6fd6e71. Read the comment docs.

@github-actions
Copy link

Benchmark for d85d8ab

Click to view benchmark
Test Base PR %
accumulator_append 607.8±30.91µs 604.9±27.46µs -0.48%
block_apply/block_apply_10 630.7±1.24ms 630.7±2.08ms 0.00%
block_apply/block_apply_1000 64.2±0.03s 64.2±0.18s 0.00%
get_with_proof/db_store 36.0±0.25µs 36.7±0.34µs +1.94%
get_with_proof/mem_store 31.4±0.21µs 32.0±0.36µs +1.91%
put_and_commit/db_store/1 95.1±4.71µs 95.5±4.76µs +0.42%
put_and_commit/db_store/10 850.1±39.39µs 853.6±39.13µs +0.41%
put_and_commit/db_store/100 7.3±0.29ms 7.3±0.27ms 0.00%
put_and_commit/db_store/5 437.1±20.06µs 438.2±20.47µs +0.25%
put_and_commit/db_store/50 3.8±0.16ms 3.8±0.15ms 0.00%
put_and_commit/mem_store/1 61.5±5.64µs 61.5±5.66µs 0.00%
put_and_commit/mem_store/10 576.0±51.79µs 573.9±46.10µs -0.36%
put_and_commit/mem_store/100 5.7±0.83ms 5.7±0.82ms 0.00%
put_and_commit/mem_store/5 289.2±24.29µs 291.7±25.61µs +0.86%
put_and_commit/mem_store/50 2.8±0.19ms 2.8±0.18ms 0.00%
query_block/query_block_in(10)_times(100) 5.5±0.24ms 5.5±0.32ms 0.00%
query_block/query_block_in(10)_times(1000) 53.8±1.18ms 54.3±2.04ms +0.93%
query_block/query_block_in(10)_times(10000) 544.3±13.22ms 536.2±12.84ms -1.49%
query_block/query_block_in(1000)_times(100) 992.1±10.91µs 951.4±7.22µs -4.10%
query_block/query_block_in(1000)_times(1000) 9.8±0.08ms 9.5±0.07ms -3.06%
query_block/query_block_in(1000)_times(10000) 98.3±1.00ms 95.1±1.20ms -3.26%
storage_transaction 1076.3±290.67µs 1074.7±279.67µs -0.15%
vm/transaction_execution/1 712.3±3.50ms 712.2±9.11ms -0.01%
vm/transaction_execution/10 176.7±0.47ms 176.6±0.27ms -0.06%
vm/transaction_execution/20 152.6±0.86ms 153.5±1.85ms +0.59%
vm/transaction_execution/5 233.5±0.43ms 233.8±0.53ms +0.13%
vm/transaction_execution/50 167.1±0.66ms 167.0±1.25ms -0.06%

@nanne007 nanne007 merged commit f43e580 into master Aug 25, 2022
@nanne007 nanne007 deleted the release/v1.12.2 branch August 25, 2022 13:20
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