Closed DavidM-D closed 3 months ago
On the following benchmark
#[test(tokio::test)]
async fn overhead_bench() -> anyhow::Result<()> {
with_multichain_nodes(MultichainConfig::default(), |ctx| {
Box::pin(async move {
panic!();
})
})
.await
}
On a warm cache before:
cargo test multichain::overhead_bench -- --exact --test-threads 1 619.91s user 25.71s system 602% cpu 1:47.19 total
After
cargo test multichain::overhead_bench -- --exact --test-threads 1 1.43s user 0.78s system 14% cpu 15.621 total
So about a 7x speedup and 400x less CPU.
success
success
Pusher: @DavidM-D, Action: pull_request
, Working Directory: `, Workflow:
Terraform Feature Env`
URL: https://mpc-recovery-leader-dev-500-7tk2cmmtcq-ue.a.run.app
success
success
Pusher: @volovyks, Action: pull_request
, Working Directory: `, Workflow:
Terraform Feature Env (Destroy)`
When rustflags change all build artifacts are invalidated, because they are present in the build of all dependencies. This PR keeps the wasm builds and the native builds separate allowing them to maintain their caches. This may break some people's workflows if they're manually building and running the integration tests.