Rename stop parameter to end

This is to align more closely with the query parameter names for the prometheus range_query API.

Author: felixbarny

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/LogicalPlanBuilder.java

@@ -1146,8 +1146,8 @@ public PlanFactory visitSampleCommand(EsqlBaseParser.SampleCommandContext ctx) {
     public PlanFactory visitPromqlCommand(EsqlBaseParser.PromqlCommandContext ctx) {
         Source source = source(ctx);
         Map<String, Expression> params = new HashMap<>();
-        String TIME = "time", START = "start", STOP = "stop", STEP = "step";
-        Set<String> allowed = Set.of(TIME, START, STOP, STEP);
+        String TIME = "time", START = "start", END = "end", STEP = "step";
+        Set<String> allowed = Set.of(TIME, START, END, STEP);
 
         if (ctx.promqlParam().isEmpty()) {
             throw new ParsingException(source(ctx), "Parameter [{}] or [{}] is required", STEP, TIME);
@@ -1175,21 +1175,21 @@ public PlanFactory visitPromqlCommand(EsqlBaseParser.PromqlCommandContext ctx) {
         // Validation logic for time parameters
         Expression time = params.get(TIME);
         Expression start = params.get(START);
-        Expression stop = params.get(STOP);
+        Expression end = params.get(END);
         Expression step = params.get(STEP);
 
-        if (time != null && (start != null || stop != null || step != null)) {
+        if (time != null && (start != null || end != null || step != null)) {
             throw new ParsingException(
                 source,
                 "Specify either [{}] for instant query or [{}}], [{}] or [{}}] for a range query",
                 TIME,
                 STEP,
                 START,
-                STOP
+                END
             );
         }
-        if ((start != null || stop != null) && step == null) {
-            throw new ParsingException(source, "[{}}] is required alongside [{}}] or [{}}]", STEP, START, STOP);
+        if ((start != null || end != null) && step == null) {
+            throw new ParsingException(source, "[{}}] is required alongside [{}}] or [{}}]", STEP, START, END);
         }
 
         // TODO: Perform type and value validation
@@ -1220,7 +1220,7 @@ public PlanFactory visitPromqlCommand(EsqlBaseParser.PromqlCommandContext ctx) {
 
         return plan -> time != null
             ? new PromqlCommand(source, plan, promqlPlan, time)
-            : new PromqlCommand(source, plan, promqlPlan, start, stop, step);
+            : new PromqlCommand(source, plan, promqlPlan, start, end, step);
     }
 
     private static ParsingException getParsingException(ParsingException pe, int promqlStartLine, int promqlStartColumn) {

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/promql/PromqlCommand.java

@@ -8,7 +8,6 @@
 package org.elasticsearch.xpack.esql.plan.logical.promql;
 
 import org.elasticsearch.common.io.stream.StreamOutput;
-import org.elasticsearch.core.TimeValue;
 import org.elasticsearch.xpack.esql.capabilities.TelemetryAware;
 import org.elasticsearch.xpack.esql.core.expression.Expression;
 import org.elasticsearch.xpack.esql.core.expression.Literal;
@@ -28,34 +27,34 @@
 public class PromqlCommand extends UnaryPlan implements TelemetryAware {
 
     private final LogicalPlan promqlPlan;
-    private final Expression start, stop, step;
+    private final Expression start, end, step;
 
     // Instant query constructor - shortcut for a range constructor
     public PromqlCommand(Source source, LogicalPlan child, LogicalPlan promqlPlan, Expression time) {
         this(source, child, promqlPlan, time, time, Literal.timeDuration(source, Duration.ZERO));
     }
 
     // Range query constructor
-    public PromqlCommand(Source source, LogicalPlan child, LogicalPlan promqlPlan, Expression start, Expression stop, Expression step) {
+    public PromqlCommand(Source source, LogicalPlan child, LogicalPlan promqlPlan, Expression start, Expression end, Expression step) {
         super(source, child);
         this.promqlPlan = promqlPlan;
         this.start = start;
-        this.stop = stop;
+        this.end = end;
         this.step = step;
     }
 
     @Override
     protected NodeInfo<PromqlCommand> info() {
-        return NodeInfo.create(this, PromqlCommand::new, child(), promqlPlan(), start(), stop(), step());
+        return NodeInfo.create(this, PromqlCommand::new, child(), promqlPlan(), start(), end(), step());
     }
 
     @Override
     public PromqlCommand replaceChild(LogicalPlan newChild) {
-        return new PromqlCommand(source(), newChild, promqlPlan(), start(), stop(), step());
+        return new PromqlCommand(source(), newChild, promqlPlan(), start(), end(), step());
     }
 
     public PromqlCommand withPromqlPlan(LogicalPlan newPromqlPlan) {
-        return new PromqlCommand(source(), child(), newPromqlPlan, start(), stop(), step());
+        return new PromqlCommand(source(), child(), newPromqlPlan, start(), end(), step());
     }
 
     @Override
@@ -86,8 +85,8 @@ public Expression start() {
         return start;
     }
 
-    public Expression stop() {
-        return stop;
+    public Expression end() {
+        return end;
     }
 
     public Expression step() {
@@ -96,7 +95,7 @@ public Expression step() {
 
     @Override
     public int hashCode() {
-        return Objects.hash(child(), start, stop, step, promqlPlan);
+        return Objects.hash(child(), start, end, step, promqlPlan);
     }
 
     @Override
@@ -114,10 +113,10 @@ public boolean equals(Object obj) {
     public String nodeString() {
         StringBuilder sb = new StringBuilder();
         sb.append(nodeName());
-        if (start == stop) {
+        if (start == end) {
             sb.append("time=").append(start);
         } else {
-            sb.append("start=").append(start).append(", stop=").append(stop).append(", step=").append(step);
+            sb.append("start=").append(start).append(", end=").append(end).append(", step=").append(step);
         }
         sb.append(" promql=[<>\n");
         sb.append(promqlPlan.toString());