Skip to content

Commit 5a84912

Browse files
committed
Add comment
1 parent 20f91c5 commit 5a84912

File tree

1 file changed

+2
-0
lines changed
  • x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/execution

1 file changed

+2
-0
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/execution/PlanExecutor.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,8 @@ public void esql(
8787
planningMetricsManager.publish(planningMetrics, false);
8888
listener.onFailure(ex);
8989
});
90+
// Wrap it in a listener so that if we have any exceptions during execution, the listener picks it up
91+
// and all the metrics are properly updated
9092
ActionListener.run(executeListener, l -> session.execute(request, executionInfo, planRunner, l));
9193
}
9294

0 commit comments

Comments
 (0)