Skip to content

[8.14] [Obs AI Assistant] Remove ES|QL escaping for index names (#183028)#183037

Merged
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-183028
May 9, 2024
Merged

[8.14] [Obs AI Assistant] Remove ES|QL escaping for index names (#183028)#183037
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-183028

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

)

There were some last minute changes to ES|QL that will ship in 8.14 that
we need to take into account:

- [BUCKET is now an aggregation
function](elastic/elasticsearch#107272)
- [index names can no longer be escaped with backticks
](elastic/elasticsearch#108431)

I'm also including a change that translates `=` to `==` in WHERE
commands, and more useful error messages (map a syntax error to the
command where it occurred).

As BUCKET is often used for timeseries data and we replaced single and
double quotes around index names with backticks, this introduces a high
chance of generating syntactically invalid queries. This PR updates the
docs and examples and removes the correction from `"` and `'` to "`"

(cherry picked from commit 6004cad)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label May 9, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) May 9, 2024 12:20
@botelastic botelastic bot added the Team:Obs AI Assistant Observability AI Assistant label May 9, 2024
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @dgieselaar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:Obs AI Assistant Observability AI Assistant

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants