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

change the rust version from 1.64.0 to 1.66.0 #3874

Merged
merged 6 commits into from
Mar 22, 2023
Merged

Conversation

jackzhhuang
Copy link
Collaborator

@jackzhhuang jackzhhuang commented Mar 17, 2023

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 Mar 17, 2023

Codecov Report

Merging #3874 (119e8c5) into master (bc5f9e3) will increase coverage by 0.13%.
The diff coverage is 33.34%.

❗ Current head 119e8c5 differs from pull request most recent head a049465. Consider uploading reports for the commit a049465 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3874      +/-   ##
==========================================
+ Coverage   53.77%   53.89%   +0.13%     
==========================================
  Files         616      616              
  Lines       67796    67804       +8     
==========================================
+ Hits        36448    36536      +88     
+ Misses      31348    31268      -80     
Flag Coverage Δ
unittests 53.89% <33.34%> (+0.13%) ⬆️

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

Impacted Files Coverage Δ
cmd/generator/src/lib.rs 3.85% <0.00%> (ø)
cmd/starcoin/src/account/import_cmd.rs 0.00% <0.00%> (ø)
commons/scmd/src/context.rs 0.00% <0.00%> (ø)
stratum/src/rpc.rs 3.06% <0.00%> (ø)
vm/resource-viewer/src/lib.rs 0.64% <0.00%> (ø)
vm/types/src/account_config/constants/account.rs 66.67% <0.00%> (ø)
vm/types/src/account_config/constants/event.rs 0.00% <0.00%> (ø)
vm/types/src/transaction/authenticator.rs 63.06% <0.00%> (ø)
account/src/account.rs 86.21% <100.00%> (ø)
commons/logger/src/structured_log.rs 8.83% <100.00%> (ø)
... and 2 more

... and 27 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@nkysg nkysg linked an issue Mar 18, 2023 that may be closed by this pull request
@sanlee42 sanlee42 force-pushed the update_to_1.65.0 branch 2 times, most recently from b999b4c to e88f9b0 Compare March 20, 2023 01:53
@github-actions
Copy link

Benchmark for 950bbb9

Click to view benchmark
Test Base PR %
accumulator_append 601.9±26.66µs 599.2±29.90µs -0.45%
block_apply/block_apply_10 272.1±6.76ms 272.2±0.49ms +0.04%
block_apply/block_apply_1000 27.7±0.03s 28.0±0.02s +1.08%
get_with_proof/db_store 36.4±0.50µs 37.3±0.22µs +2.47%
get_with_proof/mem_store 31.5±0.14µs 31.4±0.22µs -0.32%
put_and_commit/db_store/1 96.7±5.21µs 96.4±4.67µs -0.31%
put_and_commit/db_store/10 857.4±38.11µs 860.9±36.85µs +0.41%
put_and_commit/db_store/100 7.3±0.31ms 7.4±0.30ms +1.37%
put_and_commit/db_store/5 441.2±21.18µs 440.9±21.70µs -0.07%
put_and_commit/db_store/50 3.8±0.16ms 3.9±0.17ms +2.63%
put_and_commit/mem_store/1 61.7±5.77µs 62.0±5.73µs +0.49%
put_and_commit/mem_store/10 576.5±48.61µs 578.9±47.22µs +0.42%
put_and_commit/mem_store/100 5.6±0.34ms 5.7±0.83ms +1.79%
put_and_commit/mem_store/5 290.7±25.55µs 292.9±25.12µs +0.76%
put_and_commit/mem_store/50 2.8±0.19ms 2.9±0.19ms +3.57%
query_block/query_block_in(10)_times(100) 3.9±0.12ms 3.8±0.11ms -2.56%
query_block/query_block_in(10)_times(1000) 38.9±0.92ms 39.0±1.20ms +0.26%
query_block/query_block_in(10)_times(10000) 385.1±9.75ms 388.7±5.83ms +0.93%
query_block/query_block_in(1000)_times(100) 976.7±17.92µs 1002.7±5.03µs +2.66%
query_block/query_block_in(1000)_times(1000) 9.6±0.08ms 10.1±0.14ms +5.21%
query_block/query_block_in(1000)_times(10000) 96.8±2.26ms 100.3±0.76ms +3.62%
storage_transaction 1003.2±325.00µs 1126.4±372.01µs +12.28%
vm/transaction_execution/1 314.9±1.48ms 316.9±0.62ms +0.64%
vm/transaction_execution/10 99.1±1.70ms 99.1±1.03ms 0.00%
vm/transaction_execution/20 92.7±2.06ms 92.2±0.17ms -0.54%
vm/transaction_execution/5 120.3±0.80ms 121.1±0.32ms +0.67%
vm/transaction_execution/50 106.4±0.21ms 106.7±0.21ms +0.28%

@github-actions
Copy link

Benchmark for 83c6837

Click to view benchmark
Test Base PR %
accumulator_append 601.8±29.10µs 608.8±26.23µs +1.16%
block_apply/block_apply_10 270.0±0.61ms 275.2±1.72ms +1.93%
block_apply/block_apply_1000 27.8±0.08s 28.1±0.02s +1.08%
get_with_proof/db_store 36.3±0.20µs 37.1±0.26µs +2.20%
get_with_proof/mem_store 31.3±0.25µs 31.6±0.47µs +0.96%
put_and_commit/db_store/1 97.0±4.68µs 97.8±5.12µs +0.82%
put_and_commit/db_store/10 858.9±40.44µs 873.6±40.89µs +1.71%
put_and_commit/db_store/100 7.3±0.26ms 7.4±0.27ms +1.37%
put_and_commit/db_store/5 444.0±23.07µs 448.4±22.70µs +0.99%
put_and_commit/db_store/50 3.8±0.17ms 3.8±0.16ms 0.00%
put_and_commit/mem_store/1 61.8±5.89µs 62.3±5.82µs +0.81%
put_and_commit/mem_store/10 577.9±50.98µs 585.2±50.66µs +1.26%
put_and_commit/mem_store/100 5.7±0.85ms 5.8±0.90ms +1.75%
put_and_commit/mem_store/5 291.6±25.12µs 293.6±24.98µs +0.69%
put_and_commit/mem_store/50 2.8±0.19ms 2.9±0.19ms +3.57%
query_block/query_block_in(10)_times(100) 3.8±0.12ms 3.9±0.08ms +2.63%
query_block/query_block_in(10)_times(1000) 38.9±0.72ms 38.7±1.29ms -0.51%
query_block/query_block_in(10)_times(10000) 386.4±7.86ms 387.8±7.36ms +0.36%
query_block/query_block_in(1000)_times(100) 968.8±4.06µs 991.2±10.27µs +2.31%
query_block/query_block_in(1000)_times(1000) 9.7±0.07ms 10.0±0.09ms +3.09%
query_block/query_block_in(1000)_times(10000) 97.1±0.51ms 99.0±0.89ms +1.96%
storage_transaction 1031.6±310.19µs 1023.2±316.90µs -0.81%
vm/transaction_execution/1 316.2±0.75ms 319.2±1.51ms +0.95%
vm/transaction_execution/10 98.7±0.30ms 99.7±0.47ms +1.01%
vm/transaction_execution/20 92.1±0.50ms 92.5±0.44ms +0.43%
vm/transaction_execution/5 120.4±0.28ms 121.7±0.48ms +1.08%
vm/transaction_execution/50 106.9±0.53ms 107.4±0.26ms +0.47%

@@ -672,7 +672,7 @@ impl<'d> serde::de::DeserializeSeed<'d> for &TypeInstantiation {
T::Vector(layout) => Ok(match layout.as_ref() {
T::U8 => {
let bytes = <Vec<u8>>::deserialize(deserializer)?;
V::String(format!("0x{}", hex::encode(&bytes)))
V::String(format!("0x{}", hex::encode(bytes)))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里不需要传引用了,rust编译器内部做了啥?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里应该是因为bytes是Vec,而Vec本来就已经实现了AsRef这个trait,因此不需要&bytes,直接bytes传过去就好。

@nkysg nkysg self-requested a review March 21, 2023 06:35
@nkysg nkysg changed the title change the rust version from 1.64.0 to 1.65.0 change the rust version from 1.64.0 to 1.66.0 Mar 21, 2023
@nkysg nkysg merged commit 4856e37 into master Mar 22, 2023
@nkysg nkysg deleted the update_to_1.65.0 branch March 22, 2023 04:09
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.

[Dev] use rust 1.66.0
2 participants