Skip to content

Commit

Permalink
[chore] bump move version
Browse files Browse the repository at this point in the history
Take advantage of diem/move#13, which will allow us to build Move packages concurrently without encountering flakiness due to races in the cache. Split a test that caused flakiness before to demonstrate that it works.

This will also allow us to move `ObjectBasics` out of the framework and into a test package, but I don't want to do that here because there are a few in-flight PR's that touch this module.
  • Loading branch information
sblackshear committed Jan 15, 2022
1 parent ae17191 commit e51bc66
Show file tree
Hide file tree
Showing 7 changed files with 28 additions and 31 deletions.
6 changes: 3 additions & 3 deletions fastpay/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ fastpay_core = { path = "../fastpay_core" }
fastx-adapter = { path = "../fastx_programmability/adapter" }
fastx-types = { path = "../fastx_types" }

move-package = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-core-types = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-package = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-core-types = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }

[[bin]]
name = "client"
Expand Down
6 changes: 3 additions & 3 deletions fastpay_core/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,9 @@ fastx-adapter = { path = "../fastx_programmability/adapter" }
fastx-framework = { path = "../fastx_programmability/framework" }
fastx-types = { path = "../fastx_types" }

move-binary-format = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-core-types = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-vm-runtime = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-binary-format = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-core-types = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-vm-runtime = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }


typed-store = { git = "https://github.com/MystenLabs/mysten-infra", rev = "0ef3f1fedcfbf3dfe0eeea65e05de073b7c25733" }
Expand Down
14 changes: 7 additions & 7 deletions fastx_programmability/adapter/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -13,20 +13,20 @@ bcs = "0.1.3"
once_cell = "1.9.0"
structopt = "0.3.25"

move-binary-format = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-bytecode-utils = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-core-types = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-cli = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-vm-runtime = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-binary-format = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-bytecode-utils = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-core-types = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-cli = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-vm-runtime = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }

fastx-framework = { path = "../framework" }
fastx-verifier = { path = "../verifier" }
fastx-types = { path = "../../fastx_types" }

[dev-dependencies]
datatest-stable = "0.1.1"
move-package = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-package = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }

[[bin]]
name = "fastx"
Expand Down
18 changes: 9 additions & 9 deletions fastx_programmability/framework/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,15 @@ smallvec = "1.7.0"
fastx-types = { path = "../../fastx_types" }
fastx-verifier = { path = "../verifier" }

move-binary-format = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-cli = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-core-types = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-package = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-stdlib = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-unit-test = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-vm-runtime = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-vm-types = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-binary-format = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-cli = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-core-types = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-package = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-stdlib = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-unit-test = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-vm-runtime = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-vm-types = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }


[package.metadata.cargo-udeps.ignore]
Expand Down
5 changes: 1 addition & 4 deletions fastx_programmability/framework/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -58,12 +58,9 @@ fn build(sub_dir: &str) -> Vec<CompiledModule> {
#[test]
fn check_that_move_code_can_be_built_verified_tested() {
get_fastx_framework_modules();
// ideally this would be a separate test, but doing so introduces
// races because of https://github.com/diem/diem/issues/10102
run_move_unit_tests();
}

#[cfg(test)]
#[test]
fn run_move_unit_tests() {
use fastx_types::{FASTX_FRAMEWORK_ADDRESS, MOVE_STDLIB_ADDRESS};
use move_cli::package::cli::{self, UnitTestResult};
Expand Down
6 changes: 3 additions & 3 deletions fastx_programmability/verifier/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ license = "Apache-2.0"
publish = false

[dependencies]
move-binary-format = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-core-types = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-binary-format = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-bytecode-verifier = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-core-types = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }

fastx-types = { path = "../../fastx_types" }
4 changes: 2 additions & 2 deletions fastx_types/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,5 @@ thiserror = "1.0.30"
hex = "0.4.3"


move-binary-format = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-core-types = { git = "https://github.com/diem/move", rev="98ed299a7e3a9223019c9bdf4dd92fea9faef860" }
move-binary-format = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }
move-core-types = { git = "https://github.com/diem/move", rev="d253bf1c21314679d17e6935116f30baf409f264" }

1 comment on commit e51bc66

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

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

Bench results

�[0m�[0m�[1m�[32m Finished�[0m release [optimized + debuginfo] target(s) in 3.68s
�[0m�[0m�[1m�[32m Running�[0m target/release/bench
[2022-01-15T23:36:30Z INFO bench] Starting benchmark: OrdersAndCerts
[2022-01-15T23:36:30Z INFO bench] Preparing accounts.
[2022-01-15T23:36:30Z INFO bench] Open database on path: "/tmp/DB_744BBC76C9BF7739639C13BC3C154367"
[2022-01-15T23:36:34Z INFO bench] Preparing transactions.
[2022-01-15T23:36:42Z INFO fastpay::network] Listening to Tcp traffic on 127.0.0.1:9555
[2022-01-15T23:36:43Z INFO bench] Number of TCP connections: 2
[2022-01-15T23:36:43Z INFO bench] Set max_in_flight to 500
[2022-01-15T23:36:43Z INFO bench] Sending requests.
[2022-01-15T23:36:43Z INFO fastpay::network] Sending Tcp requests to 127.0.0.1:9555
[2022-01-15T23:36:44Z INFO fastpay::network] Sending Tcp requests to 127.0.0.1:9555
[2022-01-15T23:36:45Z INFO fastpay::network] 127.0.0.1:9555 has processed 5000 packets
[2022-01-15T23:36:46Z INFO fastpay::network] In flight 500 Remaining 35000
[2022-01-15T23:36:46Z INFO fastpay::network] 127.0.0.1:9555 has processed 10000 packets
[2022-01-15T23:36:48Z INFO fastpay::network] 127.0.0.1:9555 has processed 15000 packets
[2022-01-15T23:36:49Z INFO fastpay::network] In flight 500 Remaining 30000
[2022-01-15T23:36:49Z INFO fastpay::network] 127.0.0.1:9555 has processed 20000 packets
[2022-01-15T23:36:50Z INFO fastpay::network] 127.0.0.1:9555 has processed 25000 packets
[2022-01-15T23:36:51Z INFO fastpay::network] In flight 500 Remaining 25000
[2022-01-15T23:36:52Z INFO fastpay::network] In flight 500 Remaining 25000
[2022-01-15T23:36:52Z INFO fastpay::network] 127.0.0.1:9555 has processed 30000 packets
[2022-01-15T23:36:53Z INFO fastpay::network] 127.0.0.1:9555 has processed 35000 packets
[2022-01-15T23:36:54Z INFO fastpay::network] In flight 500 Remaining 20000
[2022-01-15T23:36:54Z INFO fastpay::network] 127.0.0.1:9555 has processed 40000 packets
[2022-01-15T23:36:55Z INFO fastpay::network] 127.0.0.1:9555 has processed 45000 packets
[2022-01-15T23:36:55Z INFO fastpay::network] In flight 500 Remaining 15000
[2022-01-15T23:36:55Z INFO fastpay::network] 127.0.0.1:9555 has processed 50000 packets
[2022-01-15T23:36:56Z INFO fastpay::network] 127.0.0.1:9555 has processed 55000 packets
[2022-01-15T23:36:57Z INFO fastpay::network] In flight 500 Remaining 10000
[2022-01-15T23:36:57Z INFO fastpay::network] 127.0.0.1:9555 has processed 60000 packets
[2022-01-15T23:36:58Z INFO fastpay::network] 127.0.0.1:9555 has processed 65000 packets
[2022-01-15T23:36:58Z INFO fastpay::network] In flight 500 Remaining 5000
[2022-01-15T23:36:59Z INFO fastpay::network] In flight 500 Remaining 5000
[2022-01-15T23:36:59Z INFO fastpay::network] 127.0.0.1:9555 has processed 70000 packets
[2022-01-15T23:36:59Z INFO fastpay::network] 127.0.0.1:9555 has processed 75000 packets
[2022-01-15T23:37:00Z INFO fastpay::network] Done sending Tcp requests to 127.0.0.1:9555
[2022-01-15T23:37:00Z INFO fastpay::network] 127.0.0.1:9555 has processed 80000 packets
[2022-01-15T23:37:00Z INFO bench] Received 80000 responses.
[2022-01-15T23:37:00Z WARN bench] Completed benchmark for OrdersAndCerts
Total time: 16845802us, items: 40000, tx/sec: 2374.478816740218

Please sign in to comment.