Skip to content

Commit 29a9611

Browse files
committed
gate tests requiring normal hash behavior
1 parent 20bdfa2 commit 29a9611

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

‎datafusion/physical-plan/src/joins/hash_join/partitioned_hash_eval.rs‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1017,6 +1017,9 @@ mod tests {
10171017
));
10181018
// Probe keys are hashed with the deserialized seed, so 1 and 3
10191019
// (whose hashes were serialized) match and 2 and 4 do not.
1020+
// Under force_hash_collisions every value hashes identically,
1021+
// so the filter degenerates to all-true and this doesn't hold.
1022+
#[cfg(not(feature = "force_hash_collisions"))]
10201023
assert_eq!(
10211024
eval_lookup(expr, &probe_batch(&[1, 2, 3, 4])),
10221025
[true, false, true, false]
@@ -1140,6 +1143,7 @@ mod tests {
11401143
.unwrap();
11411144

11421145
let batch = probe_batch(&[0, 1, 2, 3, 4, 5, 6]);
1146+
#[cfg(not(feature = "force_hash_collisions"))]
11431147
assert_eq!(
11441148
eval_lookup(&expr, &batch),
11451149
[false, true, false, true, false, true, false]
@@ -1222,6 +1226,7 @@ mod tests {
12221226
.unwrap();
12231227

12241228
let batch = probe_batch(&[1, 2, 3, 4, 5]);
1229+
#[cfg(not(feature = "force_hash_collisions"))]
12251230
assert_eq!(
12261231
eval_lookup(&expr, &batch),
12271232
[false, true, false, true, false]
@@ -1286,6 +1291,7 @@ mod tests {
12861291
],
12871292
)
12881293
.unwrap();
1294+
#[cfg(not(feature = "force_hash_collisions"))]
12891295
assert_eq!(eval_lookup(&expr, &batch), [true, false, false, true]);
12901296
assert_eq!(
12911297
eval_lookup(&expr, &batch),

0 commit comments

Comments
 (0)