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

[Discover] Track adhoc data views usage #144244

Merged
merged 8 commits into from
Nov 17, 2022
Merged

Conversation

dimaanj
Copy link
Contributor

@dimaanj dimaanj commented Oct 31, 2022

Summary

Closes #141090

This PR enabled metrics for adhoc data views usage in Discover.

@dimaanj dimaanj added Feature:Discover Discover Application release_note:skip Skip the PR/issue when compiling release notes Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. v8.6.0 labels Oct 31, 2022
@dimaanj dimaanj self-assigned this Oct 31, 2022
@dimaanj dimaanj marked this pull request as ready for review October 31, 2022 16:12
@dimaanj dimaanj requested a review from a team as a code owner October 31, 2022 16:12
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

trackUiMetric(METRIC_TYPE.CLICK, ADHOC_DATA_VIEW_CLICK_EVENT, adHocDataViewList.length);
}
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [adHocDataViewList]);
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess we should send an event only if current data view on Discover is ad-hoc, not for the available list of them. On Lens page it's a bit different: several data views can be active and referenced in vis layers.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed.

@dimaanj
Copy link
Contributor Author

dimaanj commented Nov 10, 2022

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

merge conflict between base and head

…dhoc-usage

# Conflicts:
#	src/plugins/discover/public/application/main/hooks/use_discover_state.ts
@dimaanj dimaanj requested a review from jughosta November 11, 2022 09:18
@dimaanj
Copy link
Contributor Author

dimaanj commented Nov 14, 2022

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
discover 414.1KB 414.3KB +227.0B

Page load bundle

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

id before after diff
discover 27.9KB 27.9KB +55.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 67 73 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

History

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

cc @dimaanj

@dimaanj dimaanj added v8.7.0 auto-backport Deprecated - use backport:version if exact versions are needed labels Nov 17, 2022
@dimaanj dimaanj merged commit 2454fd0 into elastic:main Nov 17, 2022
kibanamachine pushed a commit that referenced this pull request Nov 17, 2022
## Summary

Closes #141090

This PR enabled metrics for adhoc data views usage in Discover.

Co-authored-by: Julia Rechkunova <[email protected]>
(cherry picked from commit 2454fd0)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.6

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

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

kibanamachine added a commit that referenced this pull request Nov 17, 2022
# Backport

This will backport the following commits from `main` to `8.6`:
- [[Discover] Track adhoc data views usage
(#144244)](#144244)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Dmitry
Tomashevich","email":"[email protected]"},"sourceCommit":{"committedDate":"2022-11-17T11:29:35Z","message":"[Discover]
Track adhoc data views usage (#144244)\n\n## Summary\r\n\r\nCloses
#141090\r\n\r\nThis PR enabled metrics for adhoc data views usage in
Discover.\r\n\r\nCo-authored-by: Julia Rechkunova
<[email protected]>","sha":"2454fd01b08767b31f25fe4674ac92b9633606f6","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Discover","release_note:skip","auto-backport","Team:DataDiscovery","v8.6.0","v8.7.0"],"number":144244,"url":"https://github.com/elastic/kibana/pull/144244","mergeCommit":{"message":"[Discover]
Track adhoc data views usage (#144244)\n\n## Summary\r\n\r\nCloses
#141090\r\n\r\nThis PR enabled metrics for adhoc data views usage in
Discover.\r\n\r\nCo-authored-by: Julia Rechkunova
<[email protected]>","sha":"2454fd01b08767b31f25fe4674ac92b9633606f6"}},"sourceBranch":"main","suggestedTargetBranches":["8.6"],"targetPullRequestStates":[{"branch":"8.6","label":"v8.6.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/144244","number":144244,"mergeCommit":{"message":"[Discover]
Track adhoc data views usage (#144244)\n\n## Summary\r\n\r\nCloses
#141090\r\n\r\nThis PR enabled metrics for adhoc data views usage in
Discover.\r\n\r\nCo-authored-by: Julia Rechkunova
<[email protected]>","sha":"2454fd01b08767b31f25fe4674ac92b9633606f6"}}]}]
BACKPORT-->

Co-authored-by: Dmitry Tomashevich <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Discover Discover Application release_note:skip Skip the PR/issue when compiling release notes Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. v8.6.0 v8.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Discover] Track ad hoc data view usage
6 participants