From 6174f487d1aefc7500a09f9f2752529211000d1b Mon Sep 17 00:00:00 2001 From: Andy Leiserson Date: Tue, 3 Dec 2024 10:40:54 -0800 Subject: [PATCH] Disable the large_batch test in the multi-threading config It is much slower than in the single thread config. --- ipa-core/src/protocol/context/dzkp_validator.rs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/ipa-core/src/protocol/context/dzkp_validator.rs b/ipa-core/src/protocol/context/dzkp_validator.rs index 993b06dc9..5cc726369 100644 --- a/ipa-core/src/protocol/context/dzkp_validator.rs +++ b/ipa-core/src/protocol/context/dzkp_validator.rs @@ -1162,9 +1162,9 @@ mod tests { let a: Vec = repeat_with(|| rng.gen()).take(count).collect(); let b: Vec = repeat_with(|| rng.gen()).take(count).collect(); - // Timeout is 20 seconds plus count * (5 ms). + // Timeout is 10 seconds plus count * (3 ms). let config = TestWorldConfig::default() - .with_timeout_secs(20 + 5 * u64::try_from(count).unwrap() / 1000); + .with_timeout_secs(10 + 3 * u64::try_from(count).unwrap() / 1000); let [ab0, ab1, ab2]: [Vec>; 3] = TestWorld::::with_config(&config) @@ -1360,7 +1360,11 @@ mod tests { } } + // This test is much slower in the multi-threading config, perhaps because the + // amount of work it does for each record is very small compared to the overhead of + // spawning tasks. #[tokio::test] + #[cfg(not(feature = "multi-threading"))] async fn large_batch() { multi_select_malicious::(2 * TARGET_PROOF_SIZE, 2 * TARGET_PROOF_SIZE).await; }