Skip to content

Commit

Permalink
Disable the large_batch test in the multi-threading config
Browse files Browse the repository at this point in the history
It is much slower than in the single thread config.
  • Loading branch information
andyleiserson committed Dec 3, 2024
1 parent 135fbe1 commit 6174f48
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions ipa-core/src/protocol/context/dzkp_validator.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1162,9 +1162,9 @@ mod tests {
let a: Vec<V> = repeat_with(|| rng.gen()).take(count).collect();
let b: Vec<V> = 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<Replicated<V>>; 3] =
TestWorld::<NotSharded>::with_config(&config)
Expand Down Expand Up @@ -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::<BA8>(2 * TARGET_PROOF_SIZE, 2 * TARGET_PROOF_SIZE).await;
}
Expand Down

0 comments on commit 6174f48

Please sign in to comment.