Skip to content

Commit bde27c5

Browse files
authored
Use soft assert for missing delta join arrangements. (MaterializeInc#11723)
1 parent 62d94ef commit bde27c5

File tree

1 file changed

+2
-2
lines changed
  • src/dataflow-types/src/plan

1 file changed

+2
-2
lines changed

src/dataflow-types/src/plan/mod.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ use std::collections::BTreeMap;
1515
use std::collections::BTreeSet;
1616
use std::collections::HashMap;
1717

18+
use mz_ore::soft_panic_or_log;
1819
use serde::{Deserialize, Serialize, Serializer};
19-
use tracing::error;
2020

2121
use mz_expr::{
2222
permutation_for_arrangement, CollectionPlan, EvalError, GlobalId, Id, JoinInputMapper, LocalId,
@@ -598,7 +598,7 @@ impl<T: timely::progress::Timestamp> Plan<T> {
598598
// we shouldn't plan delta joins at all if not all of the required arrangements
599599
// are available. Print an error message, to increase the chances that
600600
// the user will tell us about this.
601-
error!("Arrangements depended on by delta join alarmingly absent: {:?}
601+
soft_panic_or_log!("Arrangements depended on by delta join alarmingly absent: {:?}
602602
This is not expected to cause incorrect results, but could indicate a performance issue in Materialize.", missing);
603603
} else {
604604
// It's fine and expected that linear joins don't have all their arrangements available up front,

0 commit comments

Comments
 (0)