[Unified Search] Move autocomplete logic to unified search plugin by nlatipov · Pull Request #129977 · elastic/kibana

nlatipov · 2022-04-12T07:07:14Z

Summary

Motivation:
After PR #127651 is merged, next step is move the autocomplete folder to the unified_search plugin. No extra functionality is going to be added (or cleanups or improvements).

What was done:

data/plugin/autocomplete folder was moved into unified_search;
data/plugin/server/autocomplete folder was moved into unified_search;
The following part was moved from from the Data Plugin runtime contract to UnifiedSearch Plugin

  autocomplete: this.autocomplete.start(),

Related Static and Runtime imports were updated
Added the unifiedSearch plugin as a dependency to plugins that require plugins (Kibana.json)
Registered a unifiedSearch plugin in the i18n (translations) plugin
data/server/config_deprecations.ts file was moved into unified_search/server and added renameFromRoot for autocomplete;
Added Autocomplete service
config_deprecations.ts file was moved from Data plugin server to UnifiedSearch server folder
Bundle size optimization will be implemented in another pull request issue 129978

elasticmachine · 2022-04-12T08:50:12Z

Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors)

azasypkin

Changes in expectedExposedConfigKeys LGTM (from security perspective).

nlatipov · 2022-04-19T16:49:12Z

@elasticmachine merge upstream

kibanamachine · 2022-04-19T16:49:14Z

merge conflict between base and head

…lete-to-unified-search

nlatipov · 2022-04-19T17:09:16Z

@banderror thanks for your review, attentiveness and comment regarding the tests. We will also take into account your recommendations in other PRs.

nlatipov · 2022-04-20T05:52:19Z

@elasticmachine merge upstream

…lete-to-unified-search

ashokaditya

plugins/security_solution changes look good.

…lete-to-unified-search

nlatipov · 2022-04-20T18:13:55Z

@pzl, @elastic/infra-monitoring-ui, @elastic/security-threat-hunting-investigations we need your 👀 here. Please review

michaelolo24

Threat Hunting Investigations changes LGTM! Thanks for doing this work

jasonrhodes

LGTM! (I hate when prettier just decides to reorder imports while you're doing other things lol)

…lete-to-unified-search

nlatipov · 2022-04-21T11:02:49Z

@elasticmachine merge upstream

nlatipov · 2022-04-21T12:16:45Z

@elasticmachine merge upstream

nlatipov · 2022-04-22T01:11:49Z

@elasticmachine merge upstream

kibanamachine · 2022-04-22T01:12:01Z

expected head sha didn’t match current head ref.

kibana-ci · 2022-04-22T02:59:04Z

💛 Build succeeded, but was flaky

Test Failures

[job] [logs] Cases Cypress Tests on Security Solution / Alerts timeline Privileges: can crud should allow a user with crud privileges to attach alerts to cases

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
`controls`	180	182	+2
`data`	470	456	-14
`unifiedSearch`	143	157	+14
total			+2

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
`data`	2811	2803	-8
`unifiedSearch`	45	75	+30
total			+22

Any counts in public APIs

Total count of every any typed public API. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats any for more detailed information.

id	before	after	diff
`data`	39	38	-1
`unifiedSearch`	1	2	+1
total			-0

Async chunks

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

id	before	after	diff
`apm`	2.8MB	2.8MB	+9.0B
`controls`	437.6KB	437.7KB	+100.0B
`fleet`	693.3KB	693.3KB	+18.0B
`infra`	1002.7KB	1002.7KB	+18.0B
`lens`	1.1MB	1.1MB	+148.0B
`ml`	3.3MB	3.3MB	+9.0B
`monitoring`	471.1KB	471.1KB	+19.0B
`securitySolution`	4.8MB	4.8MB	+36.0B
`synthetics`	783.7KB	783.7KB	+9.0B
`unifiedSearch`	138.4KB	116.2KB	-22.2KB
total			-21.8KB

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id	before	after	diff
`data`	20	18	-2
`unifiedSearch`	6	11	+5
total			+3

Page load bundle

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

id	before	after	diff
`controls`	34.1KB	34.2KB	+53.0B
`data`	423.1KB	410.9KB	-12.2KB
`inputControlVis`	5.4KB	5.5KB	+32.0B
`maps`	70.1KB	70.1KB	+9.0B
`ml`	39.3KB	39.3KB	+30.0B
`monitoring`	23.5KB	23.7KB	+143.0B
`unifiedSearch`	41.5KB	87.8KB	+46.3KB
total			+34.3KB

Unknown metric groups

API count

id	before	after	diff
`data`	3421	3415	-6
`unifiedSearch`	48	79	+31
total			+25

async chunk count

id	before	after	diff
`unifiedSearch`	10	9	-1

ESLint disabled line counts

id	before	after	diff
`data`	40	39	-1
`unifiedSearch`	12	15	+3
total			+2

References to deprecated APIs

id	before	after	diff
`data`	397	375	-22
`unifiedSearch`	145	253	+108
total			+86

Total ESLint disabled count

id	before	after	diff
`data`	44	43	-1
`unifiedSearch`	12	15	+3
total			+2

History

💔 Build #39791 failed bec9740
💔 Build #39762 failed 2c0b86e
💔 Build #39667 failed 68ab513
💔 Build #39563 failed bc59a0f
💔 Build #39394 failed 596caeb

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

cc @nlatipov

…astic#129977) * feat: move autocomplete logic from data plugin to unified search * minor fix after comments * updated Documentation: data.autocomplete -> unifiedSearch.autocomplete * changed renameFromRoot order for autocomplete * removed extra renameFromRoot in config deprecations, updated test * added configPath for unified search plugin * Update kibana.json * updated path to autocomplete * fix conflict * fix conflict * [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix' * fix Linting * fix functional_with_es_ssl test

feat: move autocomplete logic from data plugin to unified search

17ffceb

alexwizp assigned nlatipov Apr 12, 2022

nlatipov mentioned this pull request Apr 12, 2022

[Unified search] Optimise the bundle size unified search plugin #129978

Closed

nlatipov marked this pull request as ready for review April 12, 2022 08:50

nlatipov requested review from a team as code owners April 12, 2022 08:50

nlatipov requested a review from a team April 12, 2022 08:50

nlatipov requested review from a team as code owners April 12, 2022 08:50

nlatipov requested a review from pzl April 12, 2022 08:50

nlatipov requested a review from ashokaditya April 12, 2022 08:50

azasypkin approved these changes Apr 12, 2022

View reviewed changes

botelastic Bot added Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Fleet Team label for Observability Data Collection Fleet team labels Apr 12, 2022

Merge remote-tracking branch 'upstream/main' into issue-move-autocomp…

e7a6e55

…lete-to-unified-search

kibanamachine and others added 2 commits April 20, 2022 00:52

Merge branch 'main' into issue-move-autocomplete-to-unified-search

158ca32

Merge remote-tracking branch 'upstream/main' into issue-move-autocomp…

596caeb

…lete-to-unified-search

ashokaditya approved these changes Apr 20, 2022

View reviewed changes

nlatipov added 3 commits April 20, 2022 18:50

Merge remote-tracking branch 'upstream/main' into issue-move-autocomp…

934205f

…lete-to-unified-search

Merge remote-tracking branch 'upstream/main' into issue-move-autocomp…

9648d46

…lete-to-unified-search

fix functional_with_es_ssl test

bc59a0f

pzl removed their request for review April 20, 2022 18:14

michaelolo24 approved these changes Apr 20, 2022

View reviewed changes

neptunian approved these changes Apr 20, 2022

View reviewed changes

jasonrhodes approved these changes Apr 20, 2022

View reviewed changes

Merge remote-tracking branch 'upstream/main' into issue-move-autocomp…

68ab513

…lete-to-unified-search

Merge branch 'main' into issue-move-autocomplete-to-unified-search

2c0b86e

Merge branch 'main' into issue-move-autocomplete-to-unified-search

bec9740

Merge branch 'main' into issue-move-autocomplete-to-unified-search

2194981

nlatipov merged commit 7c01257 into elastic:main Apr 22, 2022

Dosant mentioned this pull request Apr 25, 2022

Remove data_enhanced plugin #122075

Merged

1 task

weltenwort mentioned this pull request Jul 27, 2022

[Infra Monitoring UI] KQL auto-completion does not work #137267

Closed

Conversation

nlatipov commented Apr 12, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

elasticmachine commented Apr 12, 2022

Uh oh!

azasypkin left a comment

Choose a reason for hiding this comment

Uh oh!

nlatipov commented Apr 19, 2022

Uh oh!

kibanamachine commented Apr 19, 2022

Uh oh!

nlatipov commented Apr 19, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nlatipov commented Apr 20, 2022

Uh oh!

ashokaditya left a comment

Choose a reason for hiding this comment

Uh oh!

nlatipov commented Apr 20, 2022

Uh oh!

michaelolo24 left a comment

Choose a reason for hiding this comment

Uh oh!

jasonrhodes left a comment

Choose a reason for hiding this comment

Uh oh!

nlatipov commented Apr 21, 2022

Uh oh!

nlatipov commented Apr 21, 2022

Uh oh!

nlatipov commented Apr 22, 2022

Uh oh!

kibanamachine commented Apr 22, 2022

Uh oh!

kibana-ci commented Apr 22, 2022

💛 Build succeeded, but was flaky

Test Failures

Metrics [docs]

Module Count

Public APIs missing comments

Any counts in public APIs

Async chunks

Public APIs missing exports

Page load bundle

API count

async chunk count

ESLint disabled line counts

References to deprecated APIs

Total ESLint disabled count

History

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

nlatipov commented Apr 12, 2022 •

edited

Loading

nlatipov commented Apr 19, 2022 •

edited

Loading