Skip to content

[ES|QL] Calculate the query for retrieving the values correctly#214905

Merged
stratoula merged 3 commits intoelastic:mainfrom
stratoula:fix-generated-query
Mar 21, 2025
Merged

[ES|QL] Calculate the query for retrieving the values correctly#214905
stratoula merged 3 commits intoelastic:mainfrom
stratoula:fix-generated-query

Conversation

@stratoula
Copy link
Contributor

@stratoula stratoula commented Mar 18, 2025

Summary

My function to calculate the query for retrieving the values wasn't complete. This fixes it by taking under consideration the cursor position

meow

Checklist

@stratoula stratoula added v9.1.0 v8.19.0 Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana t// release_note:fix backport:version Backport to applied version labels labels Mar 18, 2025
@stratoula stratoula marked this pull request as ready for review March 18, 2025 10:15
@stratoula stratoula requested a review from a team as a code owner March 18, 2025 10:15
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/esql-utils 88 89 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
esql 240.9KB 241.4KB +559.0B
Unknown metric groups

API count

id before after diff
@kbn/esql-utils 101 102 +1

History

Copy link
Contributor

@drewdaemon drewdaemon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good.

I guess we don't support looking ahead, but that's in line with everything else in autocomplete 👍

Screen.Recording.2025-03-20.at.10.02.11.AM.mov
@stratoula stratoula merged commit 96ba76d into elastic:main Mar 21, 2025
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/13986175497

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Mar 21, 2025
…tic#214905)

## Summary

My function to calculate the query for retrieving the values wasn't
complete. This fixes it by taking under consideration the cursor
position

![meow](https://github.com/user-attachments/assets/eac8bd8e-4e09-406e-969f-eb8dd95a0e61)

### Checklist
- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 96ba76d)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Mar 21, 2025
…#214905) (#215434)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] Calculate the query for retrieving the values correctly
(#214905)](#214905)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Stratoula
Kalafateli","email":"efstratia.kalafateli@elastic.co"},"sourceCommit":{"committedDate":"2025-03-21T06:44:39Z","message":"[ES|QL]
Calculate the query for retrieving the values correctly (#214905)\n\n##
Summary\n\nMy function to calculate the query for retrieving the values
wasn't\ncomplete. This fixes it by taking under consideration the
cursor\nposition\n\n\n![meow](https://github.com/user-attachments/assets/eac8bd8e-4e09-406e-969f-eb8dd95a0e61)\n\n\n###
Checklist\n- [ ] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"96ba76de2b5cccbf71c7588e01d763bb45a6d7aa","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Feature:ES|QL","Team:ESQL","backport:version","v9.1.0","v8.19.0"],"title":"[ES|QL]
Calculate the query for retrieving the values
correctly","number":214905,"url":"https://github.com/elastic/kibana/pull/214905","mergeCommit":{"message":"[ES|QL]
Calculate the query for retrieving the values correctly (#214905)\n\n##
Summary\n\nMy function to calculate the query for retrieving the values
wasn't\ncomplete. This fixes it by taking under consideration the
cursor\nposition\n\n\n![meow](https://github.com/user-attachments/assets/eac8bd8e-4e09-406e-969f-eb8dd95a0e61)\n\n\n###
Checklist\n- [ ] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"96ba76de2b5cccbf71c7588e01d763bb45a6d7aa"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/214905","number":214905,"mergeCommit":{"message":"[ES|QL]
Calculate the query for retrieving the values correctly (#214905)\n\n##
Summary\n\nMy function to calculate the query for retrieving the values
wasn't\ncomplete. This fixes it by taking under consideration the
cursor\nposition\n\n\n![meow](https://github.com/user-attachments/assets/eac8bd8e-4e09-406e-969f-eb8dd95a0e61)\n\n\n###
Checklist\n- [ ] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"96ba76de2b5cccbf71c7588e01d763bb45a6d7aa"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this pull request Mar 24, 2025
…tic#214905)

## Summary

My function to calculate the query for retrieving the values wasn't
complete. This fixes it by taking under consideration the cursor
position


![meow](https://github.com/user-attachments/assets/eac8bd8e-4e09-406e-969f-eb8dd95a0e61)


### Checklist
- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Mar 31, 2025
…tic#214905)

## Summary

My function to calculate the query for retrieving the values wasn't
complete. This fixes it by taking under consideration the cursor
position


![meow](https://github.com/user-attachments/assets/eac8bd8e-4e09-406e-969f-eb8dd95a0e61)


### Checklist
- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Feature:ES|QL ES|QL related features in Kibana release_note:fix Team:ESQL ES|QL related features in Kibana t// v8.19.0 v9.1.0

4 participants