Skip to content

[8.x] [ES|QL] Fixes the suggestion problem in where for multiline queries (#213240)#213744

Merged
stratoula merged 2 commits intoelastic:8.xfrom
stratoula:backport/8.x/pr-213240
Mar 10, 2025
Merged

[8.x] [ES|QL] Fixes the suggestion problem in where for multiline queries (#213240)#213744
stratoula merged 2 commits intoelastic:8.xfrom
stratoula:backport/8.x/pr-213240

Conversation

@stratoula
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…lastic#213240)

## Summary

Closes elastic#213323

This is the attempt to fix this bug:

```
FROM kibana_sample_data_logs
| WHERE event.dataset == # cursor on this line
| LIMIT 10
```

In main the suggestions do not trigger. The problem is that the range is
completely wrong. The lineNumber is 3 while it should be 2 and the start
and end columns are also wrong.

This PR attempts to fix it (hopefully).

![meow](https://github.com/user-attachments/assets/2741891e-5186-477b-900f-ef42bb3371da)

### 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

(cherry picked from commit 6f83177)

# Conflicts:
#	src/platform/packages/shared/kbn-monaco/src/esql/lib/shared/utils.test.ts
@stratoula stratoula added the backport This PR is a backport of another PR label Mar 10, 2025
@stratoula stratoula enabled auto-merge (squash) March 10, 2025 13:50
@stratoula
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

There are no new commits on the base branch.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-srcJs 3.6MB 3.6MB +3.0B

History

@stratoula stratoula disabled auto-merge March 10, 2025 16:32
@stratoula stratoula merged commit a986789 into elastic:8.x Mar 10, 2025
8 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants