Skip to content

Commit 4ea309f

Browse files
authored
Reinstantiate join order (#1122)
1 parent 167bbf9 commit 4ea309f

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

ballista/scheduler/src/state/execution_graph/execution_stage.rs

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ use std::sync::Arc;
2222
use std::time::{SystemTime, UNIX_EPOCH};
2323

2424
use datafusion::physical_optimizer::aggregate_statistics::AggregateStatistics;
25+
use datafusion::physical_optimizer::join_selection::JoinSelection;
2526
use datafusion::physical_optimizer::PhysicalOptimizerRule;
2627
use datafusion::physical_plan::display::DisplayableExecutionPlan;
2728
use datafusion::physical_plan::metrics::{MetricValue, MetricsSet};
@@ -359,12 +360,9 @@ impl UnresolvedStage {
359360
&input_locations,
360361
)?;
361362

362-
// TODO reinstate this logic once https://github.com/apache/datafusion/issues/10978
363-
// is fixed
364-
// Optimize join order and statistics based on new resolved statistics
365-
// let optimize_join = JoinSelection::new();
366-
// let config = SessionConfig::default();
367-
// let plan = optimize_join.optimize(plan, config.options())?;
363+
let optimize_join = JoinSelection::new();
364+
let config = SessionConfig::default();
365+
let plan = optimize_join.optimize(plan, config.options())?;
368366

369367
let optimize_aggregate = AggregateStatistics::new();
370368
let plan =

0 commit comments

Comments
 (0)