Add comment

smalyshev · Dec 12, 2024 · 5a84912 · 5a84912
1 parent 20f91c5
commit 5a84912
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/execution/PlanExecutor.java b/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/execution/PlanExecutor.java
@@ -87,6 +87,8 @@ public void esql(
             planningMetricsManager.publish(planningMetrics, false);
             listener.onFailure(ex);
         });
+        // Wrap it in a listener so that if we have any exceptions during execution, the listener picks it up
+        // and all the metrics are properly updated
         ActionListener.run(executeListener, l -> session.execute(request, executionInfo, planRunner, l));
     }