Skip to content
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] Separate GROK and DISSECT autocomplete routines #211101

Merged
merged 6 commits into from
Feb 14, 2025

Conversation

drewdaemon
Copy link
Contributor

@drewdaemon drewdaemon commented Feb 13, 2025

Summary

Part of #195418

Gives GROK and DISSECT autocomplete logic its own home 🏡

Checklist

Identify risks

  • As with any refactor, there's a possibility this will introduce a regression in the behavior of commands. However, all automated tests are passing and I have tested the behavior manually and can detect no regression.

@drewdaemon drewdaemon added Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana v9.1.0 v8.19.0 backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes labels Feb 13, 2025
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
cloudSecurityPosture 796 798 +2
controls 462 464 +2
dashboard 718 720 +2
data 627 629 +2
dataVisualizer 858 860 +2
discover 1103 1105 +2
esql 216 218 +2
esqlDataGrid 521 523 +2
eventAnnotationListing 748 750 +2
infra 1307 1309 +2
investigateApp 358 360 +2
lens 1825 1827 +2
lists 435 437 +2
maps 1332 1334 +2
ml 2437 2439 +2
observability 1447 1449 +2
observabilityAIAssistantApp 530 532 +2
securitySolution 6794 6796 +2
slo 996 998 +2
stackAlerts 278 280 +2
unifiedHistogram 290 292 +2
unifiedSearch 400 402 +2
total +44

Async chunks

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

id before after diff
securitySolution 19.1MB 19.1MB +328.0B

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 +1.0KB

@drewdaemon drewdaemon marked this pull request as ready for review February 14, 2025 00:13
@drewdaemon drewdaemon requested a review from a team as a code owner February 14, 2025 00:13
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@darnautov darnautov left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -20,7 +20,7 @@ export async function suggest({
if (/INFO\s+$/i.test(innerText)) {
return [{ ...pipeCompleteItem, command: TRIGGER_SUGGESTION_COMMAND }];
}
// SHOW INSOF /
// SHOW LOLZ /
Copy link
Contributor

Choose a reason for hiding this comment

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

lolz? 😅

Copy link
Contributor

Choose a reason for hiding this comment

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

Lolz 🤘

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lolz 😹✅

@drewdaemon drewdaemon merged commit 1223926 into elastic:main Feb 14, 2025
21 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

You might need to backport the following PRs to 8.x:
- [[ES

Manual backport

To create the backport manually run:

node scripts/backport --pr 211101

Questions ?

Please refer to the Backport tool documentation

@drewdaemon
Copy link
Contributor Author

💚 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

drewdaemon added a commit that referenced this pull request Feb 15, 2025
…11101) (#211320)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] Separate `GROK` and `DISSECT`
autocomplete routines
(#211101)](#211101)

<!--- Backport version: 9.6.4 -->

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

<!--BACKPORT [{"author":{"name":"Drew
Tate","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-14T14:44:46Z","message":"[ES|QL]
Separate `GROK` and `DISSECT` autocomplete routines (#211101)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana/issues/195418\n\nGives `GROK` and
`DISSECT` autocomplete logic its own home 🏡\n\n### Checklist\n\n- [x]
[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### Identify
risks\n\n- [ ] As with any refactor, there's a possibility this will
introduce a\nregression in the behavior of commands. However, all
automated tests are\npassing and I have tested the behavior manually and
can detect
no\nregression.","sha":"1223926450736fba5f67d17279ccf650d8b3ff2a","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:ES|QL","Team:ESQL","backport:version","v9.1.0","v8.19.0"],"title":"[ES|QL]
Separate `GROK` and `DISSECT` autocomplete
routines","number":211101,"url":"https://github.com/elastic/kibana/pull/211101","mergeCommit":{"message":"[ES|QL]
Separate `GROK` and `DISSECT` autocomplete routines (#211101)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana/issues/195418\n\nGives `GROK` and
`DISSECT` autocomplete logic its own home 🏡\n\n### Checklist\n\n- [x]
[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### Identify
risks\n\n- [ ] As with any refactor, there's a possibility this will
introduce a\nregression in the behavior of commands. However, all
automated tests are\npassing and I have tested the behavior manually and
can detect
no\nregression.","sha":"1223926450736fba5f67d17279ccf650d8b3ff2a"}},"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/211101","number":211101,"mergeCommit":{"message":"[ES|QL]
Separate `GROK` and `DISSECT` autocomplete routines (#211101)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana/issues/195418\n\nGives `GROK` and
`DISSECT` autocomplete logic its own home 🏡\n\n### Checklist\n\n- [x]
[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### Identify
risks\n\n- [ ] As with any refactor, there's a possibility this will
introduce a\nregression in the behavior of commands. However, all
automated tests are\npassing and I have tested the behavior manually and
can detect
no\nregression.","sha":"1223926450736fba5f67d17279ccf650d8b3ff2a"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
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:skip Skip the PR/issue when compiling release notes Team:ESQL ES|QL related features in Kibana v8.19.0 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants