Skip to content

Commit 9de36af

Browse files
authored
Fix optimized plan string in EsqlSession (elastic#136689)
Thanks for the catch @idegtiarenko !
1 parent 0b6338a commit 9de36af

File tree

1 file changed

+4
-3
lines changed
  • x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session

1 file changed

+4
-3
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session/EsqlSession.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -924,11 +924,12 @@ public void preOptimizedPlan(
924924
}
925925

926926
private PhysicalPlan physicalPlan(Versioned<LogicalPlan> optimizedPlan) {
927-
if (optimizedPlan.inner().optimized() == false) {
927+
LogicalPlan logicalPlan = optimizedPlan.inner();
928+
if (logicalPlan.optimized() == false) {
928929
throw new IllegalStateException("Expected optimized plan");
929930
}
930-
optimizedLogicalPlanString = optimizedPlan.toString();
931-
var plan = mapper.map(optimizedPlan);
931+
optimizedLogicalPlanString = logicalPlan.toString();
932+
PhysicalPlan plan = mapper.map(optimizedPlan);
932933
LOGGER.debug("Physical plan:\n{}", plan);
933934
return plan;
934935
}

0 commit comments

Comments
 (0)