Skip to content

ESQL: Get and list APIs should use async execution id format #127187

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
nik9000 opened this issue Apr 22, 2025 · 1 comment · May be fixed by #127472
Open

ESQL: Get and list APIs should use async execution id format #127187

nik9000 opened this issue Apr 22, 2025 · 1 comment · May be fixed by #127472
Assignees
Labels
:Analytics/ES|QL AKA ESQL >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@nik9000
Copy link
Member

nik9000 commented Apr 22, 2025

Description

The get and list APIs for ESQL should use the same async execution id format that the async API uses. You should be able to do a _query/async which returns an id and use that id immediately in the get running query API.

The API also shouldn't return node, coordinating_node, or data_node or anything. That's a concept that's not universally applicable. So:

{ "queries" : {
 "abc:1234": {    <---- use the encoding returned by async search
  "node": "abc",   <---- zap this
  "id": 1234,   <---- zap this
  "start_time_millis": 14585858875292,
  "running_time_nanos": 762794,
  "query": "FROM logs* | STATS BY hostname"
  },
 "abd:1729": {    <---- use the encoding returned by async search
  "node": "abd",   <---- zap this
  "id":1729,   <---- zap this
  "start_time_millis": 14585858823573,
  "running_time_nanos": 90231,
  "query": "FROM orders | LOOKUP country_code ON country"
  }
 } 
}
GET /_query/queries/abc:1234    <---- use the encoding returned by async search

{
  "id": 1234,   <---- zap this
  "node": "abc",   <---- zap this
  "start_time_millis": 14585858875292,
  "running_time_nanos": 762794,
  "query": "FROM logs* | STATS BY hostname"
  "coordinating_node": "oTUltX4IQMOUUVeiohTt8A"      <---- zap this
  "data_nodes" : [ "DwrYwfytxthse49X4", "i5msnbUyWlpe86e7"]      <---- zap this
}
@nik9000 nik9000 added >enhancement needs:triage Requires assignment of a team area label labels Apr 22, 2025
@nik9000 nik9000 self-assigned this Apr 22, 2025
@nik9000 nik9000 added :Analytics/ES|QL AKA ESQL and removed needs:triage Requires assignment of a team area label labels Apr 22, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Apr 22, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

GalLalouche added a commit to GalLalouche/elasticsearch that referenced this issue Apr 28, 2025
Also the individual ID, node, and data node tags.

GitHub: Resolves elastic#127187
@GalLalouche GalLalouche assigned GalLalouche and unassigned nik9000 Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants