Skip to content

Commit ac4267c

Browse files
pepijnvealamb
andcommitted
Add comment explaining why the logical plan optimizer is triggered
Co-authored-by: Andrew Lamb <[email protected]>
1 parent 5cfe8b6 commit ac4267c

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

datafusion/core/tests/tpcds_planning.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1055,6 +1055,8 @@ async fn regression_test(query_no: u8, create_physical: bool) -> Result<()> {
10551055
if create_physical {
10561056
let _ = state.create_physical_plan(&plan).await?;
10571057
} else {
1058+
// Run the logical optimizer even if we are not creating the physical plan
1059+
// to ensure it will properly succeed
10581060
let _ = state.optimize(&plan)?;
10591061
}
10601062
}

0 commit comments

Comments
 (0)