-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[ES|QL] Improve ES|QL autocomplete suggestions for case() expressions #192135
Conversation
Pinging @elastic/kibana-esql (Team:ESQL) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very very nice! I tested it locally and works great, LGTM!
@elasticmachine merge upstream |
This improves the basic case nicely. I'm not sure how far we want to take this. Worth pointing out, though, that
WDYT about these cases @qn895 @stratoula |
Yes yes we know that. I am fine with what Quynh has done so far and I don't want to complicate things at this point. I think it gives the idea. I want to add a more complicated cases example in the recommended queries task. I think we are good for now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! I suggested a couple test cases to add.
...bn-esql-validation-autocomplete/src/autocomplete/__tests__/autocomplete.suggest.eval.test.ts
Outdated
Show resolved
Hide resolved
...bn-esql-validation-autocomplete/src/autocomplete/__tests__/autocomplete.suggest.eval.test.ts
Outdated
Show resolved
Hide resolved
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]Page load bundle
History
To update your PR or re-run it, just comment with: cc @qn895 |
Summary
This PR addresses #177258 and improves autocomplete support for
case()
expressionsAfter:
case( / )
suggest any field/to boolean function in this position as first argumentcase( field /)
suggest comparison operators at this point to converge to a booleancase( field > / )
suggest field/function of the same type of the right hand side to complete the boolean expressioncase( field > 0, / )
suggest normal fields and functionsScreen.Recording.2024-09-04.at.15.59.06.mov
Screen.Recording.2024-09-05.at.12.44.02.mov
Checklist
Delete any items that are not applicable to this PR.
Risk Matrix
Delete this section if it is not applicable to this PR.
Before closing this PR, invite QA, stakeholders, and other developers to identify risks that should be tested prior to the change/feature release.
When forming the risk matrix, consider some of the following examples and how they may potentially impact the change:
For maintainers