Skip to content

Commit fb9fb84

Browse files
committed
Disable sanity check temporarily
1 parent 025ddde commit fb9fb84

1 file changed

Lines changed: 11 additions & 2 deletions

File tree

datafusion/physical-optimizer/src/sanity_checker.rs

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ use datafusion_physical_plan::{get_plan_string, ExecutionPlanProperties};
3737
use crate::PhysicalOptimizerRule;
3838
use datafusion_physical_expr_common::sort_expr::format_physical_sort_requirement_list;
3939
use itertools::izip;
40+
use log::warn;
4041

4142
/// The SanityCheckPlan rule rejects the following query plans:
4243
/// 1. Invalid plans containing nodes whose order and/or distribution requirements
@@ -59,8 +60,16 @@ impl PhysicalOptimizerRule for SanityCheckPlan {
5960
plan: Arc<dyn ExecutionPlan>,
6061
config: &ConfigOptions,
6162
) -> Result<Arc<dyn ExecutionPlan>> {
62-
plan.transform_up(|p| check_plan_sanity(p, &config.optimizer))
63-
.data()
63+
plan.transform_up(|p| {
64+
match check_plan_sanity(Arc::clone(&p), &config.optimizer) {
65+
Ok(v) => Ok(v),
66+
Err(err) => {
67+
warn!("Error during SanityCheckPlan: {err}");
68+
Ok(Transformed::no(p))
69+
}
70+
}
71+
})
72+
.data()
6473
}
6574

6675
fn name(&self) -> &str {

0 commit comments

Comments
 (0)